ACPICA: iasl/acpiexec: Update input file handling and verification
authorBob Moore <robert.moore@intel.com>
Tue, 29 Dec 2015 05:55:05 +0000 (13:55 +0800)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Fri, 1 Jan 2016 02:47:34 +0000 (03:47 +0100)
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>
drivers/acpi/acpica/acapps.h
drivers/acpi/acpica/dbinput.c
drivers/acpi/acpica/exmisc.c
drivers/acpi/acpica/tbutils.c

index 185c2eb78bf99920b58831e21e00848bed0db059..340350f55ea90377faab9fc1a5ffc2985d81c259 100644 (file)
 /* 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 */
 
index 71357febd0d5b7b12043c5d91ffc8667e8a2e5cb..802779a3d913377a49f69a7238b94f77351890b6 100644 (file)
@@ -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);
                        }
index b5413bb8b37475ba8039b52e85d921a2e2891df2..a8ce8a8778b3e982ce57fde111e64d8ef88233c1 100644 (file)
@@ -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;
 
index 72bcccf1b7c495b376fec8ee23e13954ef2cf078..7c1b5f8a5cbf2f9fc8727f64eb6b305474320924 100644 (file)
@@ -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++) {