uboot-oxnas: fix memory leak in tool mkox820crc
authorAndrea Dalla Costa <andrea@dallacosta.me>
Sat, 28 Dec 2019 16:43:40 +0000 (17:43 +0100)
committerJohn Crispin <john@phrozen.org>
Wed, 15 Jan 2020 22:15:19 +0000 (23:15 +0100)
In function `main` add calls to `free` for the variable `executable`.
This is needed because the variable `executable` is allocated but
never freed. This cause a memory leak.

Signed-off-by: Andrea Dalla Costa <andrea@dallacosta.me>
package/boot/uboot-oxnas/src/tools/mkox820crc.c

index d100191f2dc96906abad0104f5d67dbf797c6b2a..873706245bc52c49a176a756accfcd3e2879c395 100644 (file)
@@ -84,6 +84,7 @@ int main(int argc, char **argv)
 
        if (status != file_length) {
                printf("Failed to load image\n");
+               free(executable);
                return -ENOENT;
        }
 
@@ -111,6 +112,7 @@ int main(int argc, char **argv)
        status = lseek(in_file, 0, SEEK_SET);
        if (status != 0) {
                printf("failed to rewind\n");
+               free(executable);
                return 1;
        }
        len = write(in_file, &img_header, sizeof(img_header));
@@ -118,6 +120,7 @@ int main(int argc, char **argv)
        len = write(in_file, executable, file_length);
        assert(len == file_length);
        close(in_file);
+       free(executable);
 
        return 0;
 }