diff options
| author | Richard W.M. Jones <rjones@redhat.com> | 2011-06-09 09:39:54 +0100 |
|---|---|---|
| committer | Richard W.M. Jones <rjones@redhat.com> | 2011-06-13 14:22:31 +0100 |
| commit | 86ef0210523a585a4c856859d36f7f888aa6fa5d (patch) | |
| tree | b7fbfed726f9413219a1ce327ec41ce8e9ba469f /src/inspect_fs_unix.c | |
| parent | 6399b416ad79fb2727cf1054b64032d401019b0d (diff) | |
| download | libguestfs-86ef0210523a585a4c856859d36f7f888aa6fa5d.tar.gz libguestfs-86ef0210523a585a4c856859d36f7f888aa6fa5d.tar.xz libguestfs-86ef0210523a585a4c856859d36f7f888aa6fa5d.zip | |
Coverity: Don't return freed pointers from command* along error path.
If the external command failed to run, we could free up the allocated
*stdoutput and *stderror pointers, but then return those freed
pointers to the caller. The caller usually tries to print and free
*stderror, so this is a serious error.
Instead, return *stdoutput as NULL, and *stderror pointing to a
generic error message.
(cherry picked from commit 29453a58d818df24c238d0a08a68886ebe4029dd)
Diffstat (limited to 'src/inspect_fs_unix.c')
0 files changed, 0 insertions, 0 deletions