From 7b0bbd886d553c3cffc5b3eb29256b05856076ee Mon Sep 17 00:00:00 2001 From: Simon Glass Date: Wed, 16 Mar 2016 07:45:41 -0600 Subject: [PATCH] mkimage: Fix missing free() and close() in fit_build() Make sure that both the error path and normal return free the buffer and close the file. Reported-by: Coverity (CID: 138491) Signed-off-by: Simon Glass Reviewed-by: Tom Rini --- tools/fit_image.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/tools/fit_image.c b/tools/fit_image.c index e628212208..9d553d10fb 100644 --- a/tools/fit_image.c +++ b/tools/fit_image.c @@ -329,7 +329,7 @@ static int fit_build(struct image_tool_params *params, const char *fname) if (ret < 0) { fprintf(stderr, "%s: Failed to build FIT image\n", params->cmdname); - goto err; + goto err_buf; } size = ret; fd = open(fname, O_RDWR | O_CREAT | O_TRUNC | O_BINARY, 0666); @@ -346,9 +346,12 @@ static int fit_build(struct image_tool_params *params, const char *fname) goto err; } close(fd); + free(buf); return 0; err: + close(fd); +err_buf: free(buf); return -1; } -- 2.39.5