xen-changelog

[Top] [All Lists]

[Xen-changelog] [xen-4.1-testing] x86: Fix argument checking in (privile

from [Xen patchbot-4 . 1-testing]

[Permanent Link][Original]

To:	xen-changelog@xxxxxxxxxxxxxxxxxxx
Subject:	[Xen-changelog] [xen-4.1-testing] x86: Fix argument checking in (privileged) function cpu_add().
From:	Xen patchbot-4.1-testing <patchbot@xxxxxxx>
Date:	Thu, 16 Jun 2011 23:55:10 +0100
Delivery-date:	Thu, 16 Jun 2011 15:55:34 -0700
Envelope-to:	www-data@xxxxxxxxxxxxxxxxxxx
List-help:	<mailto:xen-changelog-request@lists.xensource.com?subject=help>
List-id:	BK change log <xen-changelog.lists.xensource.com>
List-post:	<mailto:xen-changelog@lists.xensource.com>
List-subscribe:	<http://lists.xensource.com/mailman/listinfo/xen-changelog>, <mailto:xen-changelog-request@lists.xensource.com?subject=subscribe>
List-unsubscribe:	<http://lists.xensource.com/mailman/listinfo/xen-changelog>, <mailto:xen-changelog-request@lists.xensource.com?subject=unsubscribe>
Reply-to:	xen-devel@xxxxxxxxxxxxxxxxxxx
Sender:	xen-changelog-bounces@xxxxxxxxxxxxxxxxxxx

# HG changeset patch
# User Keir Fraser <keir@xxxxxxx>
# Date 1308167049 -3600
# Node ID 3d1a9facbc2806aaf2bb66d34ae9a03dcb665dd3
# Parent  49ed2ca8c1a28b62e65de0590bd6945ce6e740f1
x86: Fix argument checking in (privileged) function cpu_add().

Thanks to John McDermott <john.mcdermott@xxxxxxxxxxxx> for spotting.

Signed-off-by: Keir Fraser <keir@xxxxxxx>
xen-unstable changeset:   23505:5a557fda70a9
xen-unstable date:        Fri Jun 10 08:08:44 2011 +0100
---


diff -r 49ed2ca8c1a2 -r 3d1a9facbc28 xen/arch/x86/smpboot.c
--- a/xen/arch/x86/smpboot.c    Wed Jun 15 20:43:30 2011 +0100
+++ b/xen/arch/x86/smpboot.c    Wed Jun 15 20:44:09 2011 +0100
@@ -899,7 +899,9 @@
     dprintk(XENLOG_DEBUG, "cpu_add apic_id %x acpi_id %x pxm %x\n",
             apic_id, acpi_id, pxm);
 
-    if ( acpi_id > MAX_MADT_ENTRIES || apic_id > MAX_APICS || pxm > 256 )
+    if ( (acpi_id >= MAX_MADT_ENTRIES) ||
+         (apic_id >= MAX_APICS) ||
+         (pxm >= 256) )
         return -EINVAL;
 
     if ( !cpu_hotplug_begin() )

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog

[More with this subject...]

<Prev in Thread]	Current Thread	[Next in Thread>
[Xen-changelog] [xen-4.1-testing] x86: Fix argument checking in (privileged) function cpu_add()., Xen patchbot-4 . 1-testing <=

Previous by Date:	[Xen-changelog] [xen-4.1-testing] x86/apic: fix potential Protection Fault during shutdown, Xen patchbot-4 . 1-testing
Next by Date:	[Xen-changelog] [xen-4.1-testing] x86/hvm: Crash domain rather than guest on unexpected PIO IO state, Xen patchbot-4 . 1-testing
Previous by Thread:	[Xen-changelog] [xen-4.1-testing] x86/apic: fix potential Protection Fault during shutdown, Xen patchbot-4 . 1-testing
Next by Thread:	[Xen-changelog] [xen-4.1-testing] x86/hvm: Crash domain rather than guest on unexpected PIO IO state, Xen patchbot-4 . 1-testing
Indexes:	[Date] [Thread] [Top] [All Lists]