diff options
| author | Bob Moore <robert.moore@intel.com> | 2015-12-29 05:55:05 (GMT) |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2016-01-01 02:47:34 (GMT) |
| commit | 395ec731969c4bb0cc379a64809a5adab51126dd (patch) | |
| tree | cd6d83224460afae0e5f204737a929de3c15a53e /drivers/acpi | |
| parent | 2ba7379b5abcaf2e0e7ece01b1e0c925e0057cd4 (diff) | |
| download | linux-395ec731969c4bb0cc379a64809a5adab51126dd.tar.xz | |
ACPICA: iasl/acpiexec: Update input file handling and verification
ACPICA commit 3a6f2a516dd35a4daacbc5b5144d1db763ff2cb0
Improve and cleanup verification of ACPI tables within input files.
Share more code between the disassembler and acpiexec.
This patch only affects application debugger commands, thus it is a
no-op chage for Linux kernel.
Link: https://github.com/acpica/acpica/commit/3a6f2a51
Signed-off-by: Bob Moore <robert.moore@intel.com>
Signed-off-by: Lv Zheng <lv.zheng@intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'drivers/acpi')
| -rw-r--r-- | drivers/acpi/acpica/acapps.h | 10 | ||||
| -rw-r--r-- | drivers/acpi/acpica/dbinput.c | 7 | ||||
| -rw-r--r-- | drivers/acpi/acpica/exmisc.c | 4 | ||||
| -rw-r--r-- | drivers/acpi/acpica/tbutils.c | 8 |
4 files changed, 13 insertions, 16 deletions
diff --git a/drivers/acpi/acpica/acapps.h b/drivers/acpi/acpica/acapps.h index 185c2eb..340350f 100644 --- a/drivers/acpi/acpica/acapps.h +++ b/drivers/acpi/acpica/acapps.h @@ -95,9 +95,13 @@ /* acfileio */ acpi_status -acpi_ac_get_all_tables_from_file(char *filename, - u8 get_only_aml_tables, - struct acpi_new_table_desc **return_list_head); +ac_get_all_tables_from_file(char *filename, + u8 get_only_aml_tables, + struct acpi_new_table_desc **return_list_head); + +u8 ac_is_file_binary(FILE * file); + +acpi_status ac_validate_table_header(FILE * file, long table_offset); /* Values for get_only_aml_tables */ diff --git a/drivers/acpi/acpica/dbinput.c b/drivers/acpi/acpica/dbinput.c index 71357fe..802779a 100644 --- a/drivers/acpi/acpica/dbinput.c +++ b/drivers/acpi/acpica/dbinput.c @@ -1056,10 +1056,9 @@ acpi_db_command_dispatch(char *input_buffer, struct acpi_new_table_desc *list_head = NULL; status = - acpi_ac_get_all_tables_from_file(acpi_gbl_db_args - [1], - ACPI_GET_ALL_TABLES, - &list_head); + ac_get_all_tables_from_file(acpi_gbl_db_args[1], + ACPI_GET_ALL_TABLES, + &list_head); if (ACPI_SUCCESS(status)) { acpi_db_load_tables(list_head); } diff --git a/drivers/acpi/acpica/exmisc.c b/drivers/acpi/acpica/exmisc.c index b5413bb..a8ce8a8 100644 --- a/drivers/acpi/acpica/exmisc.c +++ b/drivers/acpi/acpica/exmisc.c @@ -98,9 +98,9 @@ acpi_ex_get_object_reference(union acpi_operand_object *obj_desc, default: - ACPI_ERROR((AE_INFO, "Unknown Reference Class 0x%2.2X", + ACPI_ERROR((AE_INFO, "Invalid Reference Class 0x%2.2X", obj_desc->reference.class)); - return_ACPI_STATUS(AE_AML_INTERNAL); + return_ACPI_STATUS(AE_AML_OPERAND_TYPE); } break; diff --git a/drivers/acpi/acpica/tbutils.c b/drivers/acpi/acpica/tbutils.c index 72bcccf..7c1b5f8 100644 --- a/drivers/acpi/acpica/tbutils.c +++ b/drivers/acpi/acpica/tbutils.c @@ -389,7 +389,7 @@ next_table: * * PARAMETERS: signature - Sig string to be validated * - * RETURN: TRUE if signature is correct length and has valid characters + * RETURN: TRUE if signature is has 4 valid ACPI characters * * DESCRIPTION: Validate an ACPI table signature. * @@ -399,12 +399,6 @@ u8 acpi_is_valid_signature(char *signature) { u32 i; - /* Validate the signature length */ - - if (strlen(signature) != ACPI_NAME_SIZE) { - return (FALSE); - } - /* Validate each character in the signature */ for (i = 0; i < ACPI_NAME_SIZE; i++) { |