From c890094638880072cfa95a06cb1353f0214a11a0 Mon Sep 17 00:00:00 2001 From: David Oberhollenzer Date: Sun, 10 Nov 2019 14:56:51 +0100 Subject: nanddump: don't leak copied command line arguments For some command line flags, the argument string is copied. Simply writing over the buffer leads to a resource leak if the same flag is specified on the command line more than once. This patch adds a free() call to the old buffer before overwriting it with the new copy. Signed-off-by: David Oberhollenzer --- nand-utils/nanddump.c | 1 + 1 file changed, 1 insertion(+) diff --git a/nand-utils/nanddump.c b/nand-utils/nanddump.c index 2f167bb..841ed67 100644 --- a/nand-utils/nanddump.c +++ b/nand-utils/nanddump.c @@ -162,6 +162,7 @@ static void process_options(int argc, char * const argv[]) start_addr = simple_strtoll(optarg, &error); break; case 'f': + free(dumpfile); dumpfile = xstrdup(optarg); break; case 'l': -- cgit v1.2.3