commit:     4398c48f77ef4d1c3034549ebf5150f1173341f8
Author:     Fabian Groffen <grobian <AT> gentoo <DOT> org>
AuthorDate: Sun Jan 19 18:53:52 2020 +0000
Commit:     Fabian Groffen <grobian <AT> gentoo <DOT> org>
CommitDate: Sun Jan 19 18:53:52 2020 +0000
URL:        https://gitweb.gentoo.org/proj/portage-utils.git/commit/?id=4398c48f

qpkg: fix Coverity 206544 Dereference null return value

Signed-off-by: Fabian Groffen <grobian <AT> gentoo.org>

 qpkg.c | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/qpkg.c b/qpkg.c
index 562f9af..1eee7b1 100644
--- a/qpkg.c
+++ b/qpkg.c
@@ -262,7 +262,12 @@ qpkg_make(depend_atom *atom)
        xpaksize = st.st_size - xpaksize;
 
        /* save tbz2 tail: OOOOSTOP */
-       fp = fopen(tbz2, "a");
+       if ((fp = fopen(tbz2, "a")) == NULL) {
+               warnp("could not open '%s': %s", tbz2, strerror(errno));
+               free(buf);
+               return 1;
+       }
+
        WRITE_BE_INT32(buf, xpaksize);
        fwrite(buf, 1, 4, fp);
        fwrite("STOP", 1, 4, fp);

Reply via email to