diff options
| author | Peter Oruba <[email protected]> | 2008-08-01 12:46:46 +0200 |
|---|---|---|
| committer | Ingo Molnar <[email protected]> | 2008-08-15 17:18:46 +0200 |
| commit | 636a31781684d0f49208aa163f1a5a3a74210eb4 (patch) | |
| tree | d9004b30d6519efeb204db740533d8003cf60419 | |
| parent | d33dcb9e7d272cc3171dcc3a21049902185ab03d (diff) | |
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 <[email protected]>
Cc: Peter Oruba <[email protected]>
Cc: Tigran Aivazian <[email protected]>
Signed-off-by: Ingo Molnar <[email protected]>
| -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 39961bb83293..ad136ad99cb3 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; /* |