summaryrefslogtreecommitdiffstats
path: root/arch/ia64
diff options
context:
space:
mode:
authorTony Luck <tony.luck@intel.com>2011-01-07 09:11:55 -0800
committerTony Luck <tony.luck@intel.com>2011-01-07 09:11:55 -0800
commit05f2f274c8a8747bbfb13ac8ee0c27d5f2ad8510 (patch)
tree26626f1f78d12fbe179bfeb9b7c1fecdac1d8c93 /arch/ia64
parente7d282535c94cddc208c03b7cd0815f70d676a0e (diff)
downloadop-kernel-dev-05f2f274c8a8747bbfb13ac8ee0c27d5f2ad8510.zip
op-kernel-dev-05f2f274c8a8747bbfb13ac8ee0c27d5f2ad8510.tar.gz
[IA64] Avoid array overflow if there are too many cpus in SRAT table
acpi_numa_init() has to parse the whole SRAT table, even if the kernel wants to limit the number of cpus it will use (because the ones it is going to use may be described by entries at the end of the SRAT table). Avoid overflowing the node_cpuid array. Reported-by: Yinghai Lu <yinghai@kernel.org> Signed-off-by: Tony Luck <tony.luck@intel.com>
Diffstat (limited to 'arch/ia64')
-rw-r--r--arch/ia64/kernel/acpi.c6
1 files changed, 6 insertions, 0 deletions
diff --git a/arch/ia64/kernel/acpi.c b/arch/ia64/kernel/acpi.c
index c6c90f3..7b897b7 100644
--- a/arch/ia64/kernel/acpi.c
+++ b/arch/ia64/kernel/acpi.c
@@ -477,6 +477,12 @@ acpi_numa_processor_affinity_init(struct acpi_srat_cpu_affinity *pa)
if (!(pa->flags & ACPI_SRAT_CPU_ENABLED))
return;
+ if (srat_num_cpus >= ARRAY_SIZE(node_cpuid)) {
+ printk_once(KERN_WARNING
+ "node_cpuid[%d] is too small, may not be able to use all cpus\n",
+ ARRAY_SIZE(node_cpuid));
+ return;
+ }
pxm = get_processor_proximity_domain(pa);
/* record this node in proximity bitmap */
OpenPOWER on IntegriCloud