From: Greg King <gregdk@users.sf.net>
Date: Sat, 11 Mar 2017 20:11:15 +0000 (-0500)
Subject: Fixed a pointer test again -- but, it's better than in pull request #391.
X-Git-Tag: V2.16~31^2
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=73261ea48f84ccca6ce3608fab6a9c8802ddb874;p=cc65

Fixed a pointer test again -- but, it's better than in pull request #391.

This time, the expression compares the pointer directly, instead of comparing the pointer's target.  The new expression avoids an array underrun.
---

diff --git a/test/ref/yacc.c b/test/ref/yacc.c
index d0b9190e4..ab72e24c0 100644
--- a/test/ref/yacc.c
+++ b/test/ref/yacc.c
@@ -562,13 +562,13 @@ yylook()
 		}
 
 # ifdef LEXDEBUG
-		if (*(lsp-1) < yysvec + 1)
+		if (lsp == yylstate)
 		{
 			fprintf(yyout,"yylook:  stopped (end)\n");
 		}
 		else
 		{
-			fprintf(yyout,"yylook:  stopped at %d with\n",*(lsp-1)-yysvec-1);
+			fprintf(yyout,"yylook:  stopped at %d with:\n",*(lsp-1)-(yysvec+1));
 		}
 # endif
 		while (lsp-- > yylstate)
@@ -594,7 +594,7 @@ yylook()
 				yyleng = yylastch-yytext+1;
 				yytext[yyleng] = 0;
 # ifdef LEXDEBUG
-				fprintf(yyout,"\nyylook:  match action %d\n",*yyfnd);
+				fprintf(yyout,"yylook:  match action %d\n",*yyfnd);
 				fprintf(yyout,"yylook:  done loops: %d\n",testbreak);
 # endif
 				return(*yyfnd++);