From cf6dba2a7e026b3bf62d478aa0f76fc0105c1abc Mon Sep 17 00:00:00 2001 From: espie Date: Sat, 6 Mar 2021 08:31:42 +0000 Subject: [PATCH] put back Lst_Destroy(paths) where it belongs noticed by jsg@ --- usr.bin/make/parse.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/usr.bin/make/parse.c b/usr.bin/make/parse.c index 35ab1671060..1b29df2300b 100644 --- a/usr.bin/make/parse.c +++ b/usr.bin/make/parse.c @@ -1,4 +1,4 @@ -/* $OpenBSD: parse.c,v 1.133 2020/04/20 08:17:33 espie Exp $ */ +/* $OpenBSD: parse.c,v 1.134 2021/03/06 08:31:42 espie Exp $ */ /* $NetBSD: parse.c,v 1.29 1997/03/10 21:20:04 christos Exp $ */ /* @@ -925,7 +925,6 @@ ParseDoDependency(const char *line) /* the line to parse */ ln = Lst_Adv(ln)) Dir_AddDiri(Lst_Datum(ln), line, cp); break; - Lst_Destroy(&paths, NOFREE); } default: break; @@ -989,6 +988,11 @@ ParseDoDependency(const char *line) /* the line to parse */ } } + /* If we're not in the SPEC_PATH case the list is empty, but who + * cares ? This is cheap. + */ + Lst_Destroy(&paths, NOFREE); + if (mainNode == NULL) { /* If we have yet to decide on a main target to make, in the * absence of any user input, we want the first target on -- 2.20.1