acpica-tools/name-miscompare.patch

29 lines
1.1 KiB
Diff
Raw Normal View History

2013-08-28 19:31:28 +00:00
On big-endian machines, a test case looking for the methods _L1D and _E1D
in the same scope would fail (see tests/misc/badcode.asl:184). The names
to be compared were being treated as 32-bit ints, and not strings. Hence,
the characters were re-ordered incorrectly, mismatching the assumptions
made in the remainder of the function.
Index: src/source/compiler/aslanalyze.c
===================================================================
--- src.orig/source/compiler/aslanalyze.c
+++ src/source/compiler/aslanalyze.c
@@ -445,7 +445,7 @@ ApCheckForGpeNameConflict (
2013-08-28 19:31:28 +00:00
/* Need a null-terminated string version of NameSeg */
- ACPI_MOVE_32_TO_32 (Name, &Op->Asl.NameSeg);
+ ACPI_MOVE_NAME (Name, &Op->Asl.NameSeg);
Name[ACPI_NAME_SIZE] = 0;
/*
@@ -472,7 +472,7 @@ ApCheckForGpeNameConflict (
2013-08-28 19:31:28 +00:00
* We are now sure we have an _Lxx or _Exx.
* Create the target name that would cause collision (Flip E/L)
*/
- ACPI_MOVE_32_TO_32 (Target, Name);
+ ACPI_MOVE_NAME (Target, Name);
/* Inject opposite letter ("L" versus "E") */