| commit | 557a701c16553b0b691dbb64ef30361115a80f64 | |
| author | Thomas Renninger <trenn@suse.de> | |
| Mon, 14 Dec 2009 10:44:15 +0000 (14 11:44 +0100) | ||
| committer | Dave Jones <davej@redhat.com> | |
| Wed, 13 Jan 2010 15:55:15 +0000 (13 10:55 -0500) | ||
| tree | 1160ab1c446311f5ca3a483b322cbc9ffcbc83ef | treesnapshot (tar.gz zip) |
| parent | 292e0041c3b22c5347092152504d814119554b57 | commitdiff |
[CPUFREQ] Fix use after free of struct powernow_k8_data
Easy fix for a regression introduced in 2.6.31.
On managed CPUs the cpufreq.c core will call driver->exit(cpu) on the
managed cpus and powernow_k8 will free the core's data.
Later driver->get(cpu) function might get called trying to read out the
current freq of a managed cpu and the NULL pointer check does not work on
the freed object -> better set it to NULL.
->get() is unsigned and must return 0 as invalid frequency.
Reference:
http://bugzilla.kernel.org/show_bug.cgi?id=14391
Signed-off-by: Thomas Renninger <trenn@suse.de>
Tested-by: Michal Schmidt <mschmidt@redhat.com>
CC: stable@kernel.org
Signed-off-by: Dave Jones <davej@redhat.com>
Easy fix for a regression introduced in 2.6.31.
On managed CPUs the cpufreq.c core will call driver->exit(cpu) on the
managed cpus and powernow_k8 will free the core's data.
Later driver->get(cpu) function might get called trying to read out the
current freq of a managed cpu and the NULL pointer check does not work on
the freed object -> better set it to NULL.
->get() is unsigned and must return 0 as invalid frequency.
Reference:
http://bugzilla.kernel.org/show_bug.cgi?id=14391
Signed-off-by: Thomas Renninger <trenn@suse.de>
Tested-by: Michal Schmidt <mschmidt@redhat.com>
CC: stable@kernel.org
Signed-off-by: Dave Jones <davej@redhat.com>
| arch/x86/kernel/cpu/cpufreq/powernow-k8.c | diffblobblamehistory |