Fix off-by-one check that can lead to a memory overflow in ecpg.

Michael Meskes · Michael Meskes · commit 0ba09cc02698 · 2019-04-11T21:04:37.000+02:00
Patch by Liu Huailing &lt;liuhuailing@cn.fujitsu.com&gt;
diff --git a/src/interfaces/ecpg/preproc/pgc.l b/src/interfaces/ecpg/preproc/pgc.l
@@ -1414,7 +1414,7 @@ parse_include(void)
 
 		for (ip = include_paths; yyin == NULL && ip != NULL; ip = ip->next)
 		{
-			if (strlen(ip->path) + strlen(yytext) + 3 > MAXPGPATH)
+			if (strlen(ip->path) + strlen(yytext) + 4 > MAXPGPATH)
 			{
 				fprintf(stderr, _("Error: include path \"%s/%s\" is too long on line %d, skipping\n"), ip->path, yytext, yylineno);
 				continue;

Original file line number	Diff line number	Diff line change
`@@ -1414,7 +1414,7 @@ parse_include(void)`
`1414`	`1414`
`1415`	`1415`	`for (ip = include_paths; yyin == NULL && ip != NULL; ip = ip->next)`
`1416`	`1416`	`{`
`1417`		`- if (strlen(ip->path) + strlen(yytext) + 3 > MAXPGPATH)`
	`1417`	`+ if (strlen(ip->path) + strlen(yytext) + 4 > MAXPGPATH)`
`1418`	`1418`	`{`
`1419`	`1419`	`fprintf(stderr, _("Error: include path \"%s/%s\" is too long on line %d, skipping\n"), ip->path, yytext, yylineno);`
`1420`	`1420`	`continue;`