diff options
| author | Peter Oruba <peter.oruba@amd.com> | 2008-08-01 10:46:46 (GMT) |
|---|---|---|
| committer | Ingo Molnar <mingo@elte.hu> | 2008-08-15 15:18:46 (GMT) |
| commit | 636a31781684d0f49208aa163f1a5a3a74210eb4 (patch) | |
| tree | d9004b30d6519efeb204db740533d8003cf60419 /arch/x86/kernel/microcode.c | |
| parent | d33dcb9e7d272cc3171dcc3a21049902185ab03d (diff) | |
| download | linux-636a31781684d0f49208aa163f1a5a3a74210eb4.tar.xz | |
x86: Fixed NULL function pointer dereference in AMD microcode patch loader.
Dereference took place in code part responsible for manual installation
of microcode patches through /dev/cpu/microcode.
Signed-off-by: Peter Oruba <peter.oruba@amd.com>
Cc: Peter Oruba <peter.oruba@amd.com>
Cc: Tigran Aivazian <tigran@aivazian.fsnet.co.uk>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'arch/x86/kernel/microcode.c')
| -rw-r--r-- | arch/x86/kernel/microcode.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/arch/x86/kernel/microcode.c b/arch/x86/kernel/microcode.c index 39961bb..ad136ad 100644 --- a/arch/x86/kernel/microcode.c +++ b/arch/x86/kernel/microcode.c @@ -127,7 +127,8 @@ static int do_microcode_update(void) old = current->cpus_allowed; while ((cursor = microcode_ops->get_next_ucode(&new_mc, cursor)) > 0) { - error = microcode_ops->microcode_sanity_check(new_mc); + if (microcode_ops->microcode_sanity_check != NULL) + error = microcode_ops->microcode_sanity_check(new_mc); if (error) goto out; /* |