Module Name:    src
Committed By:   kre
Date:           Sat Jun 24 11:23:35 UTC 2017

Modified Files:
        src/bin/sh: parser.c

Log Message:
Another ancient (highly improbable) bug bites the dust.   This one
caused by incorrect macro usage (ie: using the wrong one) which has
been in the sources since version 1.1 (ie: forever).

Like the previous (STACKSTRNUL) bug, the probability of this one
actually occurring has been infinitesimal but the LINENO code increases
that to infinitesimal and a smidgen... (or a few, depending upon usage).

Still, apparently that was enough, Kamil Rytarowski discovered that the
zsh configure script (damn competition!) managed to trigger this problem.


To generate a diff of this commit:
cvs rdiff -u -r1.138 -r1.139 src/bin/sh/parser.c

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

Modified files:

Index: src/bin/sh/parser.c
diff -u src/bin/sh/parser.c:1.138 src/bin/sh/parser.c:1.139
--- src/bin/sh/parser.c:1.138	Sat Jun 17 07:22:12 2017
+++ src/bin/sh/parser.c	Sat Jun 24 11:23:35 2017
@@ -1,4 +1,4 @@
-/*	$NetBSD: parser.c,v 1.138 2017/06/17 07:22:12 kre Exp $	*/
+/*	$NetBSD: parser.c,v 1.139 2017/06/24 11:23:35 kre Exp $	*/
 
 /*-
  * Copyright (c) 1991, 1993
@@ -37,7 +37,7 @@
 #if 0
 static char sccsid[] = "@(#)parser.c	8.7 (Berkeley) 5/16/95";
 #else
-__RCSID("$NetBSD: parser.c,v 1.138 2017/06/17 07:22:12 kre Exp $");
+__RCSID("$NetBSD: parser.c,v 1.139 2017/06/24 11:23:35 kre Exp $");
 #endif
 #endif /* not lint */
 
@@ -1495,11 +1495,11 @@ parsebackq(VSS *const stack, char * cons
 	parsebackquote = savepbq;
 	handler = savehandler;
 	if (arinest || ISDBLQUOTE()) {
-		USTPUTC(CTLBACKQ | CTLQUOTE, out);
+		STPUTC(CTLBACKQ | CTLQUOTE, out);
 		while (--lno >= 0)
-			USTPUTC(CTLNONL, out);
+			STPUTC(CTLNONL, out);
 	} else
-		USTPUTC(CTLBACKQ, out);
+		STPUTC(CTLBACKQ, out);
 
 	return out;
 }

Reply via email to