CVS commit: src/sys/external/bsd/acpica/dist/compiler

2017-05-04 Thread Christos Zoulas
Module Name:src
Committed By:   christos
Date:   Thu May  4 17:17:28 UTC 2017

Modified Files:
src/sys/external/bsd/acpica/dist/compiler: asltree.c

Log Message:
Arrays can't be NULL


To generate a diff of this commit:
cvs rdiff -u -r1.9 -r1.10 src/sys/external/bsd/acpica/dist/compiler/asltree.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/external/bsd/acpica/dist/compiler/asltree.c
diff -u src/sys/external/bsd/acpica/dist/compiler/asltree.c:1.9 src/sys/external/bsd/acpica/dist/compiler/asltree.c:1.10
--- src/sys/external/bsd/acpica/dist/compiler/asltree.c:1.9	Sun Apr 30 19:13:36 2017
+++ src/sys/external/bsd/acpica/dist/compiler/asltree.c	Thu May  4 13:17:28 2017
@@ -187,18 +187,15 @@ TrAllocateNode (
 }
 
 Gbl_CommentState.Latest_Parse_Node = Op;
-if (Gbl_CommentState.Latest_Parse_Node->Asl.ParseOpName)
-{
-CvDbgPrint ("trallocatenode=Set latest parse node to this node.\n");
-CvDbgPrint ("   Op->Asl.ParseOpName = %s\n",
-Gbl_CommentState.Latest_Parse_Node->Asl.ParseOpName);
-CvDbgPrint ("   Op->Asl.ParseOpcode = 0x%x\n", ParseOpcode);
-
-if (Op->Asl.FileChanged)
-{
-CvDbgPrint("file has been changed!\n");
-}
-}
+	CvDbgPrint ("trallocatenode=Set latest parse node to this node.\n");
+	CvDbgPrint ("   Op->Asl.ParseOpName = %s\n",
+	Gbl_CommentState.Latest_Parse_Node->Asl.ParseOpName);
+	CvDbgPrint ("   Op->Asl.ParseOpcode = 0x%x\n", ParseOpcode);
+
+	if (Op->Asl.FileChanged)
+	{
+	CvDbgPrint("file has been changed!\n");
+	}
 
 /*
  * if this parse op's syntax uses () and {} (i.e. Package(1){0x00}) then



CVS commit: src/sys/external/bsd/acpica/dist/compiler

2017-02-06 Thread Christos Zoulas
Module Name:src
Committed By:   christos
Date:   Mon Feb  6 23:12:25 UTC 2017

Modified Files:
src/sys/external/bsd/acpica/dist/compiler: aslutils.c

Log Message:
Remove __DATE__! Intel really likes non-reprodible builds!


To generate a diff of this commit:
cvs rdiff -u -r1.16 -r1.17 \
src/sys/external/bsd/acpica/dist/compiler/aslutils.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/external/bsd/acpica/dist/compiler/aslutils.c
diff -u src/sys/external/bsd/acpica/dist/compiler/aslutils.c:1.16 src/sys/external/bsd/acpica/dist/compiler/aslutils.c:1.17
--- src/sys/external/bsd/acpica/dist/compiler/aslutils.c:1.16	Wed Jan 25 08:38:21 2017
+++ src/sys/external/bsd/acpica/dist/compiler/aslutils.c	Mon Feb  6 18:12:25 2017
@@ -402,7 +402,7 @@ UtDisplaySummary (
 /* Compiler name and version number */
 
 FlPrintFile (FileId, "%s version %X [%s]\n\n",
-ASL_COMPILER_NAME, (UINT32) ACPI_CA_VERSION, __DATE__);
+ASL_COMPILER_NAME, (UINT32) ACPI_CA_VERSION, "2017-01-19");
 }
 
 /* Summary of main input and output files */



CVS commit: src/sys/external/bsd/acpica/dist/compiler

2016-12-18 Thread Christos Zoulas
Module Name:src
Committed By:   christos
Date:   Sun Dec 18 16:34:19 UTC 2016

Modified Files:
src/sys/external/bsd/acpica/dist/compiler: asloptions.c

Log Message:
Fix repro build.


To generate a diff of this commit:
cvs rdiff -u -r1.1.1.8 -r1.2 \
src/sys/external/bsd/acpica/dist/compiler/asloptions.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/external/bsd/acpica/dist/compiler/asloptions.c
diff -u src/sys/external/bsd/acpica/dist/compiler/asloptions.c:1.1.1.8 src/sys/external/bsd/acpica/dist/compiler/asloptions.c:1.2
--- src/sys/external/bsd/acpica/dist/compiler/asloptions.c:1.1.1.8	Fri Nov 11 14:16:14 2016
+++ src/sys/external/bsd/acpica/dist/compiler/asloptions.c	Sun Dec 18 11:34:19 2016
@@ -70,8 +70,13 @@ AslDoResponseFile (
 #define ASL_TOKEN_SEPARATORS" \t\n"
 #define ASL_SUPPORTED_OPTIONS   "@:a:b|c|d^D:e:f^gh^i|I:l^m:no|p:P^r:s|t|T+G^v^w|x:z"
 
+#ifdef ACPI_REPRO
+static char ASL_BUILD_DATE[] = "Jan 1, 1970";
+static char ASL_BUILD_TIME[] = "00:00:00";
+#else
 static char ASL_BUILD_DATE[] = __DATE__;
 static char ASL_BUILD_TIME[] = __TIME__;
+#endif
 
 
 /***



CVS commit: src/sys/external/bsd/acpica/dist/compiler

2016-10-09 Thread Christos Zoulas
Module Name:src
Committed By:   christos
Date:   Sun Oct  9 14:06:11 UTC 2016

Modified Files:
src/sys/external/bsd/acpica/dist/compiler: aslopt.c

Log Message:
PR/51540: Henning Petersen: replace , with ;


To generate a diff of this commit:
cvs rdiff -u -r1.1.1.8 -r1.2 \
src/sys/external/bsd/acpica/dist/compiler/aslopt.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/external/bsd/acpica/dist/compiler/aslopt.c
diff -u src/sys/external/bsd/acpica/dist/compiler/aslopt.c:1.1.1.8 src/sys/external/bsd/acpica/dist/compiler/aslopt.c:1.2
--- src/sys/external/bsd/acpica/dist/compiler/aslopt.c:1.1.1.8	Wed May  4 14:15:55 2016
+++ src/sys/external/bsd/acpica/dist/compiler/aslopt.c	Sun Oct  9 10:06:11 2016
@@ -133,7 +133,7 @@ OptSearchToRoot (
  * not match, and we cannot use this optimization.
  */
 Path = &(((char *) TargetPath->Pointer)[
-TargetPath->Length - ACPI_NAME_SIZE]),
+TargetPath->Length - ACPI_NAME_SIZE]);
 ScopeInfo.Scope.Node = CurrentNode;
 
 /* Lookup the NameSeg using SEARCH_PARENT (search-to-root) */



CVS commit: src/sys/external/bsd/acpica/dist/compiler

2015-08-25 Thread Christos Zoulas
Module Name:src
Committed By:   christos
Date:   Wed Aug 26 01:58:55 UTC 2015

Modified Files:
src/sys/external/bsd/acpica/dist/compiler: dttable.c

Log Message:
CID 1298707:  Change confusing code that makes coverity think that Length
is uninitalized (we want length to be zero so instead of setting it to 0,
we check if it not zero and then we set it if it is not?!?!)


To generate a diff of this commit:
cvs rdiff -u -r1.5 -r1.6 src/sys/external/bsd/acpica/dist/compiler/dttable.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/external/bsd/acpica/dist/compiler/dttable.c
diff -u src/sys/external/bsd/acpica/dist/compiler/dttable.c:1.5 src/sys/external/bsd/acpica/dist/compiler/dttable.c:1.6
--- src/sys/external/bsd/acpica/dist/compiler/dttable.c:1.5	Tue Aug 18 06:39:46 2015
+++ src/sys/external/bsd/acpica/dist/compiler/dttable.c	Tue Aug 25 21:58:55 2015
@@ -3474,7 +3474,7 @@ DtCompileGeneric (
 
 /* Now we can actually compile the parse tree */
 
-if (Length  *Length)
+if (Length)
 {
 *Length = 0;
 }



CVS commit: src/sys/external/bsd/acpica/dist/compiler

2015-08-25 Thread Christos Zoulas
Module Name:src
Committed By:   christos
Date:   Wed Aug 26 02:01:55 UTC 2015

Modified Files:
src/sys/external/bsd/acpica/dist/compiler: aslopcodes.c

Log Message:
CID 1320410: Don't call free on uninitalized pointer.


To generate a diff of this commit:
cvs rdiff -u -r1.3 -r1.4 \
src/sys/external/bsd/acpica/dist/compiler/aslopcodes.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/external/bsd/acpica/dist/compiler/aslopcodes.c
diff -u src/sys/external/bsd/acpica/dist/compiler/aslopcodes.c:1.3 src/sys/external/bsd/acpica/dist/compiler/aslopcodes.c:1.4
--- src/sys/external/bsd/acpica/dist/compiler/aslopcodes.c:1.3	Tue Aug 18 06:39:46 2015
+++ src/sys/external/bsd/acpica/dist/compiler/aslopcodes.c	Tue Aug 25 22:01:55 2015
@@ -1314,7 +1314,6 @@ OpcDoPld (
 Node = Node-Asl.Next;
 }
 
-UtLocalFree (Buffer, ACPI_PLD_BUFFER_SIZE);
 Buffer = OpcEncodePldBuffer(PldInfo);
 
 /* Change Op to a Buffer */



CVS commit: src/sys/external/bsd/acpica/dist/compiler

2015-05-09 Thread Christos Zoulas
Module Name:src
Committed By:   christos
Date:   Sat May  9 15:24:56 UTC 2015

Modified Files:
src/sys/external/bsd/acpica/dist/compiler: aslcompiler.h aslopcodes.c
aslutils.c

Log Message:
CID 1256503: Handle memory leak


To generate a diff of this commit:
cvs rdiff -u -r1.1.1.6 -r1.2 \
src/sys/external/bsd/acpica/dist/compiler/aslcompiler.h
cvs rdiff -u -r1.1.1.5 -r1.2 \
src/sys/external/bsd/acpica/dist/compiler/aslopcodes.c
cvs rdiff -u -r1.10 -r1.11 \
src/sys/external/bsd/acpica/dist/compiler/aslutils.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/external/bsd/acpica/dist/compiler/aslcompiler.h
diff -u src/sys/external/bsd/acpica/dist/compiler/aslcompiler.h:1.1.1.6 src/sys/external/bsd/acpica/dist/compiler/aslcompiler.h:1.2
--- src/sys/external/bsd/acpica/dist/compiler/aslcompiler.h:1.1.1.6	Mon Apr 13 13:21:03 2015
+++ src/sys/external/bsd/acpica/dist/compiler/aslcompiler.h	Sat May  9 11:24:56 2015
@@ -945,6 +945,11 @@ UtLocalCalloc (
 UINT32  Size);
 
 void
+UtLocalFree (
+void		*Allocated,
+UINT32  Size);
+
+void
 UtPrintFormattedName (
 UINT16  ParseOpcode,
 UINT32  Level);

Index: src/sys/external/bsd/acpica/dist/compiler/aslopcodes.c
diff -u src/sys/external/bsd/acpica/dist/compiler/aslopcodes.c:1.1.1.5 src/sys/external/bsd/acpica/dist/compiler/aslopcodes.c:1.2
--- src/sys/external/bsd/acpica/dist/compiler/aslopcodes.c:1.1.1.5	Mon Apr 13 13:21:04 2015
+++ src/sys/external/bsd/acpica/dist/compiler/aslopcodes.c	Sat May  9 11:24:56 2015
@@ -1358,6 +1358,7 @@ OpcDoPld (
 Node = Node-Asl.Next;
 }
 
+UtLocalFree (Buffer, ACPI_PLD_BUFFER_SIZE);
 Buffer = OpcEncodePldBuffer(PldInfo);
 
 /* Change Op to a Buffer */

Index: src/sys/external/bsd/acpica/dist/compiler/aslutils.c
diff -u src/sys/external/bsd/acpica/dist/compiler/aslutils.c:1.10 src/sys/external/bsd/acpica/dist/compiler/aslutils.c:1.11
--- src/sys/external/bsd/acpica/dist/compiler/aslutils.c:1.10	Mon Apr 13 13:23:15 2015
+++ src/sys/external/bsd/acpica/dist/compiler/aslutils.c	Sat May  9 11:24:56 2015
@@ -178,6 +178,26 @@ UtLocalCalloc (
 return (Allocated);
 }
 
+/***
+ *
+ * FUNCTION:UtLocalFree
+ *
+ * PARAMETERS:  Allocated   - Pointer to be released
+ * PARAMETERS:  Size- Bytes to be released
+ *
+ * RETURN:  None
+ *
+ * DESCRIPTION: Free memory previously allocated
+ *
+ **/
+void 
+UtLocalFree (
+	void *Allocated, UINT32 Size)
+{
+ACPI_FREE (Allocated);
+TotalAllocations--;
+TotalAllocated -= Size;
+}
 
 /***
  *



CVS commit: src/sys/external/bsd/acpica/dist/compiler

2014-03-23 Thread Matthias Scheler
Module Name:src
Committed By:   tron
Date:   Sun Mar 23 14:24:34 UTC 2014

Modified Files:
src/sys/external/bsd/acpica/dist/compiler: dtcompile.c

Log Message:
Make this compile.


To generate a diff of this commit:
cvs rdiff -u -r1.2 -r1.3 \
src/sys/external/bsd/acpica/dist/compiler/dtcompile.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/external/bsd/acpica/dist/compiler/dtcompile.c
diff -u src/sys/external/bsd/acpica/dist/compiler/dtcompile.c:1.2 src/sys/external/bsd/acpica/dist/compiler/dtcompile.c:1.3
--- src/sys/external/bsd/acpica/dist/compiler/dtcompile.c:1.2	Sat Mar 22 21:42:44 2014
+++ src/sys/external/bsd/acpica/dist/compiler/dtcompile.c	Sun Mar 23 14:24:34 2014
@@ -211,7 +211,7 @@ DtInitialize (
 Gbl_RootTable = NULL;
 Gbl_SubtableStack = NULL;
 
-snprintf (VersionString, sizeof(VersionBuffer), %X,
+snprintf (VersionString, sizeof(VersionString), %X,
 	(UINT32) ACPI_CA_VERSION);
 return (AE_OK);
 }



CVS commit: src/sys/external/bsd/acpica/dist/compiler

2014-03-23 Thread Matthias Scheler
Module Name:src
Committed By:   tron
Date:   Sun Mar 23 14:44:55 UTC 2014

Modified Files:
src/sys/external/bsd/acpica/dist/compiler: prutils.c

Log Message:
Make this compile.


To generate a diff of this commit:
cvs rdiff -u -r1.2 -r1.3 src/sys/external/bsd/acpica/dist/compiler/prutils.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/external/bsd/acpica/dist/compiler/prutils.c
diff -u src/sys/external/bsd/acpica/dist/compiler/prutils.c:1.2 src/sys/external/bsd/acpica/dist/compiler/prutils.c:1.3
--- src/sys/external/bsd/acpica/dist/compiler/prutils.c:1.2	Sat Mar 22 21:42:44 2014
+++ src/sys/external/bsd/acpica/dist/compiler/prutils.c	Sun Mar 23 14:44:55 2014
@@ -298,7 +298,7 @@ PrOpenIncludeFile (
 /* We could not open the include file after trying very hard */
 
 ErrorExit:
-snprintf (Gbl_MainTokenBuffer, ASL_LINE_BUFFER_SIZE, %s, %s,
+snprintf (Gbl_MainTokenBuffer, ASL_DEFAULT_LINE_BUFFER_SIZE, %s, %s,
 	Filename, strerror (errno));
 PrError (ASL_ERROR, ASL_MSG_INCLUDE_FILE_OPEN, 0);
 }



CVS commit: src/sys/external/bsd/acpica/dist/compiler

2014-01-18 Thread Alan Barrett
Module Name:src
Committed By:   apb
Date:   Sat Jan 18 09:25:24 UTC 2014

Modified Files:
src/sys/external/bsd/acpica/dist/compiler: aslutils.c

Log Message:
Use ACPI_DATE instead of __DATE__ in a version string that
ends up in the iasl executable.  ACPI_DATE is defined in
sys/external/bsd/acpica/dist/include/acapps.h.


To generate a diff of this commit:
cvs rdiff -u -r1.6 -r1.7 src/sys/external/bsd/acpica/dist/compiler/aslutils.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/external/bsd/acpica/dist/compiler/aslutils.c
diff -u src/sys/external/bsd/acpica/dist/compiler/aslutils.c:1.6 src/sys/external/bsd/acpica/dist/compiler/aslutils.c:1.7
--- src/sys/external/bsd/acpica/dist/compiler/aslutils.c:1.6	Fri Dec 27 18:50:56 2013
+++ src/sys/external/bsd/acpica/dist/compiler/aslutils.c	Sat Jan 18 09:25:24 2014
@@ -454,7 +454,7 @@ UtDisplaySummary (
 /* Compiler name and version number */
 
 FlPrintFile (FileId, %s version %X%s [%s]\n\n,
-ASL_COMPILER_NAME, (UINT32) ACPI_CA_VERSION, ACPI_WIDTH, __DATE__);
+ASL_COMPILER_NAME, (UINT32) ACPI_CA_VERSION, ACPI_WIDTH, ACPI_DATE);
 }
 
 /* Summary of main input and output files */



CVS commit: src/sys/external/bsd/acpica/dist/compiler

2013-12-30 Thread Christos Zoulas
Module Name:src
Committed By:   christos
Date:   Mon Dec 30 22:32:45 UTC 2013

Modified Files:
src/sys/external/bsd/acpica/dist/compiler: aslcompiler.y dtparser.y
prparser.y

Log Message:
Now that our yacc supports providing yytname[] and does the name protection
automatically (as it should, unlike bison), elide the name protection hacks.


To generate a diff of this commit:
cvs rdiff -u -r1.1.1.4 -r1.2 \
src/sys/external/bsd/acpica/dist/compiler/aslcompiler.y
cvs rdiff -u -r1.1.1.2 -r1.2 \
src/sys/external/bsd/acpica/dist/compiler/dtparser.y
cvs rdiff -u -r1.1.1.1 -r1.2 \
src/sys/external/bsd/acpica/dist/compiler/prparser.y

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/external/bsd/acpica/dist/compiler/aslcompiler.y
diff -u src/sys/external/bsd/acpica/dist/compiler/aslcompiler.y:1.1.1.4 src/sys/external/bsd/acpica/dist/compiler/aslcompiler.y:1.2
--- src/sys/external/bsd/acpica/dist/compiler/aslcompiler.y:1.1.1.4	Fri Dec 27 13:46:05 2013
+++ src/sys/external/bsd/acpica/dist/compiler/aslcompiler.y	Mon Dec 30 17:32:45 2013
@@ -72,7 +72,9 @@ void *  AslLocalAllo
 #define static
 #undef alloca
 #define alloca  AslLocalAllocate
+#ifndef yytname
 #define yytname AslCompilername
+#endif
 
 #define YYINITDEPTH 600 /* State stack depth */
 #define YYDEBUG 1   /* Enable debug output */

Index: src/sys/external/bsd/acpica/dist/compiler/dtparser.y
diff -u src/sys/external/bsd/acpica/dist/compiler/dtparser.y:1.1.1.2 src/sys/external/bsd/acpica/dist/compiler/dtparser.y:1.2
--- src/sys/external/bsd/acpica/dist/compiler/dtparser.y:1.1.1.2	Fri Dec 27 13:46:09 2013
+++ src/sys/external/bsd/acpica/dist/compiler/dtparser.y	Mon Dec 30 17:32:45 2013
@@ -58,7 +58,9 @@ UINT64  DtParserResu
 
 /* Bison/yacc configuration */
 
+#ifndef yytname
 #define yytname DtParsername
+#endif
 #define YYDEBUG 1   /* Enable debug output */
 #define YYERROR_VERBOSE 1   /* Verbose error messages */
 #define YYFLAG  -32768

Index: src/sys/external/bsd/acpica/dist/compiler/prparser.y
diff -u src/sys/external/bsd/acpica/dist/compiler/prparser.y:1.1.1.1 src/sys/external/bsd/acpica/dist/compiler/prparser.y:1.2
--- src/sys/external/bsd/acpica/dist/compiler/prparser.y:1.1.1.1	Fri Dec 27 13:46:10 2013
+++ src/sys/external/bsd/acpica/dist/compiler/prparser.y	Mon Dec 30 17:32:45 2013
@@ -57,7 +57,9 @@ UINT64  PrParserResu
 
 /* Bison/yacc configuration */
 
+#ifndef yytname
 #define yytname PrParsername
+#endif
 #define YYDEBUG 1   /* Enable debug output */
 #define YYERROR_VERBOSE 1   /* Verbose error messages */
 #define YYFLAG  -32768



CVS commit: src/sys/external/bsd/acpica/dist/compiler

2013-02-08 Thread Christos Zoulas
Module Name:src
Committed By:   christos
Date:   Sat Feb  9 01:22:28 UTC 2013

Modified Files:
src/sys/external/bsd/acpica/dist/compiler: aslutils.c

Log Message:
CVS/97: missing va_end()


To generate a diff of this commit:
cvs rdiff -u -r1.4 -r1.5 src/sys/external/bsd/acpica/dist/compiler/aslutils.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/external/bsd/acpica/dist/compiler/aslutils.c
diff -u src/sys/external/bsd/acpica/dist/compiler/aslutils.c:1.4 src/sys/external/bsd/acpica/dist/compiler/aslutils.c:1.5
--- src/sys/external/bsd/acpica/dist/compiler/aslutils.c:1.4	Tue Jun 28 05:04:29 2011
+++ src/sys/external/bsd/acpica/dist/compiler/aslutils.c	Fri Feb  8 20:22:28 2013
@@ -357,7 +357,6 @@ DbgPrint (
 va_list Args;
 
 
-va_start (Args, Fmt);
 
 if (!Gbl_DebugFlag)
 {
@@ -370,6 +369,7 @@ DbgPrint (
 return;
 }
 
+va_start (Args, Fmt);
 (void) vfprintf (stderr, Fmt, Args);
 va_end (Args);
 return;



CVS commit: src/sys/external/bsd/acpica/dist/compiler

2013-02-08 Thread Christos Zoulas
Module Name:src
Committed By:   christos
Date:   Sat Feb  9 01:46:30 UTC 2013

Modified Files:
src/sys/external/bsd/acpica/dist/compiler: asltree.c

Log Message:
CID/979998: missing va_end()


To generate a diff of this commit:
cvs rdiff -u -r1.1.1.2 -r1.2 \
src/sys/external/bsd/acpica/dist/compiler/asltree.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/external/bsd/acpica/dist/compiler/asltree.c
diff -u src/sys/external/bsd/acpica/dist/compiler/asltree.c:1.1.1.2 src/sys/external/bsd/acpica/dist/compiler/asltree.c:1.2
--- src/sys/external/bsd/acpica/dist/compiler/asltree.c:1.1.1.2	Thu Feb 17 05:01:02 2011
+++ src/sys/external/bsd/acpica/dist/compiler/asltree.c	Fri Feb  8 20:46:30 2013
@@ -728,6 +728,7 @@ TrLinkChildren (
 {
 AslError (ASL_WARNING, ASL_MSG_COMPILER_INTERNAL, Child,
 Child node list invalid);
+	va_end(ap);
 return Op;
 }