acpi_pad: "processor_aggregator" name too long
authorDan Carpenter <[email protected]>
Tue, 27 Apr 2010 21:06:05 +0000 (14:06 -0700)
committerLen Brown <[email protected]>
Thu, 6 May 2010 06:38:23 +0000 (02:38 -0400)
cpi_device_class can only be 19 characters and a NULL terminator.

With the current name we get a buffer overflow in acpi_pad_add()
strcpy(acpi_device_class(device), ACPI_PROCESSOR_AGGREGATOR_CLASS);

[[email protected]: call it acpi_pad, per Shaohua Li]
Signed-off-by: Dan Carpenter <[email protected]>
Cc: walter harms <[email protected]>
Acked-by: Shaohua Li <[email protected]>
Cc: Len Brown <[email protected]>
Acked-by: Pavel Machek <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Len Brown <[email protected]>
drivers/acpi/acpi_pad.c

index 19dacfd43163759df8aae2d4e018502cbc19fb1c..62122134693b808f866e9b0cea0b7349bb1ac5a0 100644 (file)
@@ -31,7 +31,7 @@
 #include <acpi/acpi_bus.h>
 #include <acpi/acpi_drivers.h>
 
-#define ACPI_PROCESSOR_AGGREGATOR_CLASS        "processor_aggregator"
+#define ACPI_PROCESSOR_AGGREGATOR_CLASS        "acpi_pad"
 #define ACPI_PROCESSOR_AGGREGATOR_DEVICE_NAME "Processor Aggregator"
 #define ACPI_PROCESSOR_AGGREGATOR_NOTIFY 0x80
 static DEFINE_MUTEX(isolated_cpus_lock);