diff options
author | Richard W.M. Jones <rjones@redhat.com> | 2011-03-07 15:55:56 +0000 |
---|---|---|
committer | Richard W.M. Jones <rjones@redhat.com> | 2011-03-07 16:42:45 +0000 |
commit | 7c721e4fd674c409b3eee60fe237d480afa1c5e2 (patch) | |
tree | acdcd920eb77e75da7a71018ae3f40838061bbed /cat/virt-filesystems.c | |
parent | 2a9e345090ceeae5bdab68407bf1133c040c4f45 (diff) | |
download | libguestfs-7c721e4fd674c409b3eee60fe237d480afa1c5e2.tar.gz libguestfs-7c721e4fd674c409b3eee60fe237d480afa1c5e2.tar.xz libguestfs-7c721e4fd674c409b3eee60fe237d480afa1c5e2.zip |
Fix trace segfault for non-daemon functions (RHBZ#682756).
Previously we expanded the code for 'trace_return' unconditionally for
all non-daemon functions. However this code was not prepared to
handle all error conditions, and in fact would segfault if it tried to
print RStringList or RHashtable where r == NULL.
We need to make the code conditional on the return value, calling
either 'trace_return' or 'trace_return_error' as appropriate.
Note the difficult case for RConstOptString which returns NULL in
non-error cases.
Diffstat (limited to 'cat/virt-filesystems.c')
0 files changed, 0 insertions, 0 deletions