Re: [libvirt PATCH 1/2] qemu: fix memory leak reported by coverity
On 10/23/20 5:40 PM, Jonathon Jongsma wrote:
Let g_autoptr clean up on early return.
Signed-off-by: Jonathon Jongsma <jjongsma(a)redhat.com>
---
src/qemu/qemu_monitor_json.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/qemu/qemu_monitor_json.c b/src/qemu/qemu_monitor_json.c
index cba9ec7b19..2491cbf9b8 100644
--- a/src/qemu/qemu_monitor_json.c
+++ b/src/qemu/qemu_monitor_json.c
@@ -4013,7 +4013,7 @@ int qemuMonitorJSONAddFileHandleToSet(qemuMonitorPtr mon,
const char *opaque,
qemuMonitorAddFdInfoPtr fdinfo)
{
- virJSONValuePtr args = NULL;
+ g_autoptr(virJSONValue) args = NULL;
g_autoptr(virJSONValue) reply = NULL;
g_autoptr(virJSONValue) cmd = NULL;
@@ -4024,7 +4024,8 @@ int qemuMonitorJSONAddFileHandleToSet(qemuMonitorPtr mon,
if (virJSONValueObjectAdd(args, "j:fdset-id", fdset, NULL) < 0)
return -1;
- if (!(cmd = qemuMonitorJSONMakeCommandInternal("add-fd", args)))
+ if (!(cmd = qemuMonitorJSONMakeCommandInternal("add-fd",
+ g_steal_pointer(&args))))
return -1;
The examples about curly braces in the coding guidelines say that curly
braces around the body in this case are optional, but the text itself
says that they are required. I'm not sure why this ambiguity exists, but
because combining the two lines of the conditional only increases the
line length to 86 (based on the sentiments expressed during a recent
discussion on that topic on the mailing list), I'm going to do that
instead of adding the curly braces.
Reviewed-by: Laine Stump <laine(a)redhat.com>
and pushed.
if (qemuMonitorJSONCommandWithFd(mon, cmd, fd, &reply) < 0)