From a4b640549df3216282f9d9924901b1521c3e2d69 Mon Sep 17 00:00:00 2001 From: Pádraig Brady
Date: Fri, 4 Sep 2015 14:38:06 +0100 Subject: maint: avoid "definitely lost" valgrind warnings Since commit v8.23-19-g8defcee, main() will return, rather than call exit(), this inducing "definitely lost" warnings in valgrind's leak checker. That precludes using the following to flag memory leaks: valgrind --leak-check=full --error-exitcode=1 \ --errors-for-leak-kinds=definite * src/pr.c (main): In dev builds, explicitly free memory allocated. * src/sort.c (main): Likewise. * src/tail.c (main): Likewise. * src/tsort.c (tsort): Likewise. --- src/tsort.c | 2 ++ 1 file changed, 2 insertions(+) (limited to 'src/tsort.c') diff --git a/src/tsort.c b/src/tsort.c index d4be03d42..0533451da 100644 --- a/src/tsort.c +++ b/src/tsort.c @@ -528,6 +528,8 @@ tsort (const char *file) } } + IF_LINT (free (root)); + if (fclose (stdin) != 0) error (EXIT_FAILURE, errno, "%s", is_stdin ? _("standard input") : quote (file)); -- cgit v1.2.3-70-g09d2