On Thu, 2006-05-11 at 10:56 -0700, Moore, Robert wrote:
> May have gotten lost in the noise, please send to me.
Attached are two patches:
1) Fixes compiler nameclash if IO keyword is used as field declaration
2) If (generic, dsdt.dsl) output file is already created with
insufficient
privileges to the current user, iasl -d dsdt segfaults.
This patch makes iasl to exit gracefully.
Thomas
Fixes compiler nameclash if IO keyword is used as field declaration
Signed-off-by: Thomas Renninger <trenn@xxxxxxx>
--- acpica-unix-20050624/compiler/aslcompiler.y.orig 2005-09-06 16:46:21.000000000 +0200
+++ acpica-unix-20050624/compiler/aslcompiler.y 2005-09-06 15:40:37.000000000 +0200
@@ -2951,6 +2951,7 @@
NameString
: NameSeg {}
| PARSEOP_NAMESTRING {$$ = TrCreateValuedLeafNode (PARSEOP_NAMESTRING, (ACPI_NATIVE_INT) AslCompilerlval.s);}
+ | PARSEOP_IO {$$ = TrCreateValuedLeafNode (PARSEOP_IO, (ACPI_NATIVE_INT) "IO");}
;
NameSeg
If (generic, dsdt.dsl) output file is already created with insufficient
privileges to the current user, iasl -d xy segfaults.
This patch fixes that.
Reference: https://bugzilla.novell.com/show_bug.cgi?id=152112
Signed-off-by: Thomas Renninger <trenn@xxxxxxx>
acpica-unix-20060127/common/adisasm.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
Index: acpica-unix-20060127/common/adisasm.c
===================================================================
--- acpica-unix-20060127/common/adisasm.c
+++ acpica-unix-20060127/common/adisasm.c
@@ -125,6 +125,7 @@
#include "acapps.h"
#include <stdio.h>
+#include <stdlib.h>
#include <string.h>
#include <time.h>
@@ -549,9 +550,8 @@ AdAmlDisassemble (
File = fopen (DisasmFilename, "w+");
if (!File)
{
- fprintf (stderr, "Could not open output file\n");
- Status = AE_ERROR;
- goto Cleanup;
+ fprintf (stderr, "Could not open output file %s\n", DisasmFilename);
+ exit (EXIT_FAILURE);
}
AcpiOsRedirectOutput (File);
