diff options
| author |   Fabian Groffen <grobian@gentoo.org> | 2020-01-19 17:00:34 +0100 |
|---|---|---|
| committer | Fabian Groffen <grobian@gentoo.org> | 2020-01-19 17:00:34 +0100 |
| commit | 1ebb60136546f233982386df561f4a83cb4d434c (patch) | |
| tree | 3a3f07095010a071223dcba0437589cdbf1e532e /qmanifest.c | |
| parent | qmerge: fix Coverity 206561 Resource leak (diff) | |
| download | portage-utils-1ebb60136546f233982386df561f4a83cb4d434c.tar.gz portage-utils-1ebb60136546f233982386df561f4a83cb4d434c.tar.bz2 portage-utils-1ebb60136546f233982386df561f4a83cb4d434c.zip | |
qmanifest: fix Coverity 206565 Unchecked return value
Signed-off-by: Fabian Groffen <grobian@gentoo.org>
Diffstat (limited to 'qmanifest.c')
| -rw-r--r-- | qmanifest.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/qmanifest.c b/qmanifest.c index 597357b..d9452a4 100644 --- a/qmanifest.c +++ b/qmanifest.c @@ -220,8 +220,8 @@ write_hashes( if (m != NULL) fwrite(data, len, 1, m); - if (gm != NULL) - gzwrite(gm, data, len); + if (gm != NULL && gzwrite(gm, data, len) == 0) + fprintf(stderr, "failed to write to compressed stream\n"); } /** @@ -430,7 +430,11 @@ generate_dir(const char *dir, enum type_manifest mtype) "IGNORE lost+found\n" "IGNORE packages\n" "IGNORE snapshots\n"); - gzwrite(mf, path, len); + if (gzwrite(mf, path, len) == 0) { + fprintf(stderr, "failed to write to file '%s/%s': %s\n", + dir, str_manifest_files_gz, strerror(errno)); + return NULL; + } if (list_dir(&dentries, &dentrieslen, dir) != 0) return NULL; |