[PATCH] tools: fix handle leak in ifdtool.c

[ant . v . moryakov]

From: Anton Moryakov <ant.v.morya...@gmail.com>

Prevent file descriptor leaks by properly closing 'fd' and 'new_fd'
when fstat() or write() operations fail.

- Added close(fd) before return in open_for_read() if fstat() fails.
- Added close(new_fd) before return in write_image() if write() fails.
- No close needed if open() fails (fd == -1 is invalid).

Signed-off-by: Anton Moryakov <ant.v.morya...@gmail.com>
---
 tools/ifdtool.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/tools/ifdtool.c b/tools/ifdtool.c
index b70570361f4..5db6fa7d195 100644
--- a/tools/ifdtool.c
+++ b/tools/ifdtool.c
@@ -499,8 +499,10 @@ static int write_image(char *filename, char *image, int 
size)
                      S_IWUSR | S_IRGRP | S_IROTH);
        if (new_fd < 0)
                return perror_fname("Could not open file '%s'", filename);
-       if (write(new_fd, image, size) != size)
+       if (write(new_fd, image, size) != size) {
+               close(new_fd);
                return perror_fname("Could not write file '%s'", filename);
+       }
        close(new_fd);
 
        return 0;
@@ -604,8 +606,10 @@ int open_for_read(const char *fname, int *sizep)
 
        if (fd == -1)
                return perror_fname("Could not open file '%s'", fname);
-       if (fstat(fd, &buf) == -1)
+       if (fstat(fd, &buf) == -1) {
+               close(fd); 
                return perror_fname("Could not stat file '%s'", fname);
+       }
        *sizep = buf.st_size;
        debug("File %s is %d bytes\n", fname, *sizep);
 
-- 
2.30.2