From: Andrea Dalla Costa <andrea@dallacosta.me>
Date: Sat, 11 Jan 2020 21:41:31 +0000 (+0100)
Subject: firmware-utils: fix possible memory leak and resource leak
X-Git-Url: http://git.openwrt.org/?a=commitdiff_plain;h=77806ad75a01458eb7ee4fcd0178b7f1600830fd;p=project%2Ffirmware-utils.git

firmware-utils: fix possible memory leak and resource leak

Add missing calls to `free` for variable `buffer`.
This could lead to a memory leak.

Add missing call to `close` for file pointer `fdin`.
This could lead to a resource leak.

Signed-off-by: Andrea Dalla Costa <andrea@dallacosta.me>
---

diff --git a/src/dns313-header.c b/src/dns313-header.c
index e69e57e..3c72b09 100644
--- a/src/dns313-header.c
+++ b/src/dns313-header.c
@@ -168,11 +168,14 @@ int main(int argc, char **argv)
 	fdin = open(pathin, O_RDONLY);
 	if (!fdin) {
 		printf("ERROR: could not open input file\n");
+		free(buffer);
 		return 0;
 	}
 	bytes = read(fdin, buffer + HEADER_SIZE, filesize);
 	if (bytes < filesize) {
 		printf("ERROR: could not read entire file\n");
+		free(buffer);
+		close(fdin);
 		return 0;
 	}
 	close(fdin);