aboutsummaryrefslogtreecommitdiff
path: root/qpkg.c
diff options
context:
space:
mode:
authorFabian Groffen <grobian@gentoo.org>2020-01-22 20:49:59 +0100
committerFabian Groffen <grobian@gentoo.org>2020-01-22 20:49:59 +0100
commitd99cc22a91b06d53cc3ef3ff159f25c71d62e750 (patch)
treeb1b5c52e67acdcfcf387f1f82044e931f3a6c5d2 /qpkg.c
parentqlop: fix Coverity 125912 Out-of-bounds access (diff)
downloadportage-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.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/qpkg.c b/qpkg.c
index 7835cd73..948d564b 100644
--- a/qpkg.c
+++ b/qpkg.c
@@ -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;
}