diff options
| author | Richard W.M. Jones <rjones@redhat.com> | 2012-03-08 14:36:25 +0000 |
|---|---|---|
| committer | Richard W.M. Jones <rjones@redhat.com> | 2012-03-08 14:36:25 +0000 |
| commit | 3f3b08a6bc610ca0886e02972257dfcb633a9d40 (patch) | |
| tree | 4e330dc344f051c95c9a0834171da9c4f9fc9e4c /src | |
| parent | b2c1d8be39d4689267ef7e4188e2cf936b30628f (diff) | |
| download | libguestfs-3f3b08a6bc610ca0886e02972257dfcb633a9d40.tar.gz libguestfs-3f3b08a6bc610ca0886e02972257dfcb633a9d40.tar.xz libguestfs-3f3b08a6bc610ca0886e02972257dfcb633a9d40.zip | |
dbdump: Avoid double-close on error path (found by Coverity).
Error: USE_AFTER_FREE:
/builddir/build/BUILD/libguestfs-1.16.5/src/dbdump.c:132: freed_arg: "pclose" frees "pp".
/builddir/build/BUILD/libguestfs-1.16.5/src/dbdump.c:142: deref_arg: Calling "pclose" dereferences freed pointer "pp".
Diffstat (limited to 'src')
| -rw-r--r-- | src/dbdump.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/dbdump.c b/src/dbdump.c index ca035db1..8242dee2 100644 --- a/src/dbdump.c +++ b/src/dbdump.c @@ -131,6 +131,7 @@ guestfs___read_db_dump (guestfs_h *g, /* Catch errors from the db_dump command. */ if (pclose (pp) != 0) { perrorf (g, "pclose: %s", cmd); + pp = NULL; goto out; } pp = NULL; |