diff options
author | Fabian Groffen <grobian@gentoo.org> | 2022-02-12 19:15:40 +0100 |
---|---|---|
committer | Fabian Groffen <grobian@gentoo.org> | 2022-02-12 19:15:40 +0100 |
commit | 219ba2dd9f748c6065c6485020f7de56623e1bc9 (patch) | |
tree | a9980b752507132fa5cd9c00c897372e0a8c6923 | |
parent | qlop: fix Coverity condition always true CID 248869 (diff) | |
download | portage-utils-219ba2dd9f748c6065c6485020f7de56623e1bc9.tar.gz portage-utils-219ba2dd9f748c6065c6485020f7de56623e1bc9.tar.bz2 portage-utils-219ba2dd9f748c6065c6485020f7de56623e1bc9.zip |
qmerge: fix Coverity file descriptor leak CID 207952
Leak would happen when opening file succeed, but statting it would fail.
Unlikely, but better to take into account anyway.
Signed-off-by: Fabian Groffen <grobian@gentoo.org>
-rw-r--r-- | qmerge.c | 3 |
1 files changed, 2 insertions, 1 deletions
@@ -1693,8 +1693,9 @@ unlink_empty(const char *buf) if (fd != -1 && stat(buf, &st) != -1) { if (st.st_size == 0) ret = unlink(buf); - close(fd); } + if (fd != -1) + close(fd); return ret; } |