x86: Fix argument checking in (privileged) function cpu_add().

Thanks to John McDermott <john.mcdermott@nrl.navy.mil> for spotting.

Signed-off-by: Keir Fraser <keir@xen.org>
This commit is contained in:
Keir Fraser committed 2011-06-10 08:08:44 +01:00
1 parent 89c9f78710
commit 3d94875683
1 file changed
+3 -1
+3 -1
View File
@@ -887,7 +887,9 @@ int cpu_add(uint32_t apic_id, uint32_t acpi_id, uint32_t pxm)
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() )
Reference in new issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.