lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 14 Aug 2008 13:55:54 +0200 From: Peter Oruba <peter.oruba@....com> To: Andrew Morton <akpm@...ux-foundation.org> CC: Ingo Molnar <mingo@...e.hu>, Thomas Gleixner <tglx@...utronix.de>, Tigran Aivazian <tigran@...azian.fsnet.co.uk>, "H. Peter Anvin" <hpa@...or.com>, LKML <linux-kernel@...r.kernel.org>, stable@...nel.org Subject: Re: [patch 2/2] [PATCH 2/2] x86: Fixed NULL function pointer dereference in AMD microcode patch loader. Andrew, that patch is only relevant in combination with the AMD microcode patch loader. Thanks, Peter Andrew Morton schrieb: > On Fri, 1 Aug 2008 12:46:46 +0200 Peter Oruba <peter.oruba@....com> wrote: > >> Dereference took place in code part responsible for manual installation >> of microcode patches through /dev/cpu/microcode. >> > > That's a bit too terse. > >> --- >> 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; >> /* > > The patch is no longer applicable to current sources. > > If the bug is sufficiently serious to warrant fixing in 2.6.25.x and in > 2.6.26.x then please prepare patches against those kernels, including > sufficient description to enable the -stable maintainers to understand > why they need to merge it. Cc those patches to stable@...nel.org. > > Thanks. > > > -- | AMD Saxony Limited Liability Company & Co. KG Operating | Wilschdorfer Landstr. 101, 01109 Dresden, Germany System | Register Court Dresden: HRA 4896 Research | General Partner authorized to represent: Center | AMD Saxony LLC (Wilmington, Delaware, US) | General Manager of AMD Saxony LLC: Dr. Hans-R. Deppe, Thomas McCoy -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists