diff options
author | 2020-01-22 20:49:59 +0100 | |
---|---|---|
committer | 2020-01-22 20:49:59 +0100 | |
commit | d99cc22a91b06d53cc3ef3ff159f25c71d62e750 (patch) | |
tree | b1b5c52e67acdcfcf387f1f82044e931f3a6c5d2 /qpkg.c | |
parent | qlop: fix Coverity 125912 Out-of-bounds access (diff) | |
download | portage-utils-d99cc22a91b06d53cc3ef3ff159f25c71d62e750.tar.gz portage-utils-d99cc22a91b06d53cc3ef3ff159f25c71d62e750.tar.bz2 portage-utils-d99cc22a91b06d53cc3ef3ff159f25c71d62e750.zip |
qpkg: fix Coverity 125923 Resource leak
Signed-off-by: Fabian Groffen <grobian@gentoo.org>
Diffstat (limited to 'qpkg.c')
-rw-r--r-- | qpkg.c | 3 |
1 files changed, 3 insertions, 0 deletions
@@ -198,18 +198,21 @@ qpkg_make(depend_atom *atom) snprintf(tmpdir, sizeof(tmpdir), "%s/qpkg.XXXXXX", qpkg_bindir); if ((i = mkstemp(tmpdir)) == -1) { + fclose(fp); free(buf); return -2; } close(i); unlink(tmpdir); if (mkdir(tmpdir, 0750)) { + fclose(fp); free(buf); return -3; } snprintf(filelist, sizeof(filelist), "%s/filelist", tmpdir); if ((out = fopen(filelist, "w")) == NULL) { + fclose(fp); free(buf); return -4; } |