diff options
author | Lin Ming <ming.m.lin@intel.com> | 2008-04-10 11:06:43 -0400 |
---|---|---|
committer | Len Brown <len.brown@intel.com> | 2008-04-22 15:35:19 -0400 |
commit | 7a5bb9964512c5313af19310c6a3002ec54f7336 (patch) | |
tree | 2508740652210f8115c7237e8763238d909106ef /drivers/acpi/utilities | |
parent | 0ba7d25c70699cdd3e06fc049d8884ee54b9d5db (diff) |
ACPICA: Fix to handle NULL package elements correctly
Fixed problem where NULL package elements were not returned to
the AcpiEvaluateObject interface correctly. Instead of returning a
NULL ACPI_OBJECT package element, the element was simply ignored,
potentially causing a buffer overflow and/or confusing the caller
who expected a fixed number of elements.
http://bugzilla.kernel.org/show_bug.cgi?id=10132
Signed-off-by: Lin Ming <ming.m.lin@intel.com>
Signed-off-by: Bob Moore <robert.moore@intel.com>
Signed-off-by: Alexey Starikovskiy <astarikovskiy@suse.de>
Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'drivers/acpi/utilities')
-rw-r--r-- | drivers/acpi/utilities/utobject.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/acpi/utilities/utobject.c b/drivers/acpi/utilities/utobject.c index 1eccd3db876f..cdb8ff5b92d1 100644 --- a/drivers/acpi/utilities/utobject.c +++ b/drivers/acpi/utilities/utobject.c | |||
@@ -470,9 +470,8 @@ acpi_ut_get_simple_object_size(union acpi_operand_object *internal_object, | |||
470 | case ACPI_TYPE_PROCESSOR: | 470 | case ACPI_TYPE_PROCESSOR: |
471 | case ACPI_TYPE_POWER: | 471 | case ACPI_TYPE_POWER: |
472 | 472 | ||
473 | /* | 473 | /* No extra data for these types */ |
474 | * No extra data for these types | 474 | |
475 | */ | ||
476 | break; | 475 | break; |
477 | 476 | ||
478 | case ACPI_TYPE_LOCAL_REFERENCE: | 477 | case ACPI_TYPE_LOCAL_REFERENCE: |