Commit 755142eaf116858c71b8f3838057537739289e5a

Stefan Sperling 2018-04-02T19:31:10

in diffreg.c, return value from preadline() must be freed

diff --git a/lib/diffreg.c b/lib/diffreg.c
index 1762aa7..157aa95 100644
--- a/lib/diffreg.c
+++ b/lib/diffreg.c
@@ -983,7 +983,6 @@ restart:
 	if (args->diff_format != D_IFDEF && a > b && c > d)
 		return (0);
 	if (args->ignore_pats != NULL) {
-		char *line;
 		/*
 		 * All lines in the change, insert, or delete must
 		 * match an ignore pattern for the change to be
@@ -991,24 +990,30 @@ restart:
 		 */
 		if (a <= b) {		/* Changes and deletes. */
 			for (i = a; i <= b; i++) {
-				line = preadline(fileno(f1),
+				char *line = preadline(fileno(f1),
 				    ds->ixold[i] - ds->ixold[i - 1],
 				    ds->ixold[i - 1]);
 				if (line == NULL)
 					return (-1);
-				if (!ignoreline(line))
+				if (!ignoreline(line)) {
+					free(line);
 					goto proceed;
+				}
+				free(line);
 			}
 		}
 		if (a > b || c <= d) {	/* Changes and inserts. */
 			for (i = c; i <= d; i++) {
-				line = preadline(fileno(f2),
+				char *line = preadline(fileno(f2),
 				    ds->ixnew[i] - ds->ixnew[i - 1],
 				    ds->ixnew[i - 1]);
 				if (line == NULL)
 					return (-1);
-				if (!ignoreline(line))
+				if (!ignoreline(line)) {
+					free(line);
 					goto proceed;
+				}
+				free(line);
 			}
 		}
 		return (0);