diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2013-05-09 19:33:42 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2013-05-09 19:33:42 -0400 |
commit | 2e99f3a12b20ab3afad0e042cc0bdd0ee855dca0 (patch) | |
tree | 611bd8acc0c14137a0e15835df52be2249ae3b35 | |
parent | fc72053bb497e3054d6e4d13ab425fc2311442d4 (diff) | |
parent | 04a29a15520463e072b1a98d8989f04233ee4a7e (diff) |
Merge tag 'acpi-fixes-3.10-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm
Pull ACPICA fixes from Rafael Wysocki:
- _INI regression fix from Tomasz Nowicki.
- Fix for a possible memory leak in _OSI support routine from Jung-uk
Kim.
- Fix for a possible buffer overflow during field unit read operation
from Bob Moore.
* tag 'acpi-fixes-3.10-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm:
ACPICA: ACPICA: Fix for _INI regression
ACPICA: _OSI support: Fix possible memory leak
ACPICA: Fix possible buffer overflow during a field unit read operation
-rw-r--r-- | drivers/acpi/acpica/exfldio.c | 14 | ||||
-rw-r--r-- | drivers/acpi/acpica/nsinit.c | 1 | ||||
-rw-r--r-- | drivers/acpi/acpica/utosi.c | 3 |
3 files changed, 16 insertions, 2 deletions
diff --git a/drivers/acpi/acpica/exfldio.c b/drivers/acpi/acpica/exfldio.c index ec7f5690031b..c84ee956fa4c 100644 --- a/drivers/acpi/acpica/exfldio.c +++ b/drivers/acpi/acpica/exfldio.c | |||
@@ -720,7 +720,19 @@ acpi_ex_extract_from_field(union acpi_operand_object *obj_desc, | |||
720 | 720 | ||
721 | if ((obj_desc->common_field.start_field_bit_offset == 0) && | 721 | if ((obj_desc->common_field.start_field_bit_offset == 0) && |
722 | (obj_desc->common_field.bit_length == access_bit_width)) { | 722 | (obj_desc->common_field.bit_length == access_bit_width)) { |
723 | status = acpi_ex_field_datum_io(obj_desc, 0, buffer, ACPI_READ); | 723 | if (buffer_length >= sizeof(u64)) { |
724 | status = | ||
725 | acpi_ex_field_datum_io(obj_desc, 0, buffer, | ||
726 | ACPI_READ); | ||
727 | } else { | ||
728 | /* Use raw_datum (u64) to handle buffers < 64 bits */ | ||
729 | |||
730 | status = | ||
731 | acpi_ex_field_datum_io(obj_desc, 0, &raw_datum, | ||
732 | ACPI_READ); | ||
733 | ACPI_MEMCPY(buffer, &raw_datum, buffer_length); | ||
734 | } | ||
735 | |||
724 | return_ACPI_STATUS(status); | 736 | return_ACPI_STATUS(status); |
725 | } | 737 | } |
726 | 738 | ||
diff --git a/drivers/acpi/acpica/nsinit.c b/drivers/acpi/acpica/nsinit.c index 2a431ec50a25..46f0f83417a1 100644 --- a/drivers/acpi/acpica/nsinit.c +++ b/drivers/acpi/acpica/nsinit.c | |||
@@ -558,6 +558,7 @@ acpi_ns_init_one_device(acpi_handle obj_handle, | |||
558 | ACPI_DEBUG_EXEC(acpi_ut_display_init_pathname | 558 | ACPI_DEBUG_EXEC(acpi_ut_display_init_pathname |
559 | (ACPI_TYPE_METHOD, device_node, METHOD_NAME__INI)); | 559 | (ACPI_TYPE_METHOD, device_node, METHOD_NAME__INI)); |
560 | 560 | ||
561 | ACPI_MEMSET(info, 0, sizeof(struct acpi_evaluate_info)); | ||
561 | info->prefix_node = device_node; | 562 | info->prefix_node = device_node; |
562 | info->pathname = METHOD_NAME__INI; | 563 | info->pathname = METHOD_NAME__INI; |
563 | info->parameters = NULL; | 564 | info->parameters = NULL; |
diff --git a/drivers/acpi/acpica/utosi.c b/drivers/acpi/acpica/utosi.c index b15acebb96a1..7e807725c636 100644 --- a/drivers/acpi/acpica/utosi.c +++ b/drivers/acpi/acpica/utosi.c | |||
@@ -349,7 +349,8 @@ acpi_status acpi_ut_osi_implementation(struct acpi_walk_state * walk_state) | |||
349 | return_value = 0; | 349 | return_value = 0; |
350 | status = acpi_os_acquire_mutex(acpi_gbl_osi_mutex, ACPI_WAIT_FOREVER); | 350 | status = acpi_os_acquire_mutex(acpi_gbl_osi_mutex, ACPI_WAIT_FOREVER); |
351 | if (ACPI_FAILURE(status)) { | 351 | if (ACPI_FAILURE(status)) { |
352 | return (status); | 352 | acpi_ut_remove_reference(return_desc); |
353 | return_ACPI_STATUS(status); | ||
353 | } | 354 | } |
354 | 355 | ||
355 | /* Lookup the interface in the global _OSI list */ | 356 | /* Lookup the interface in the global _OSI list */ |