From af3262e26f8e0022b8aefb2925aa7692ef431dd7 Mon Sep 17 00:00:00 2001 From: John Ferlan Date: Wed, 9 Jan 2013 09:54:12 -0500 Subject: [PATCH] rpc: Avoid resource leak of 'socks' if any object append fails --- src/rpc/virnetserverservice.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/rpc/virnetserverservice.c b/src/rpc/virnetserverservice.c index b39f0ef275..2e024e1d08 100644 --- a/src/rpc/virnetserverservice.c +++ b/src/rpc/virnetserverservice.c @@ -356,9 +356,6 @@ virJSONValuePtr virNetServerServicePreExecRestart(virNetServerServicePtr svc) if (!object) return NULL; - if (!(socks = virJSONValueNewArray())) - goto error; - if (virJSONValueObjectAppendNumberInt(object, "auth", svc->auth) < 0) goto error; if (virJSONValueObjectAppendBoolean(object, "readonly", svc->readonly) < 0) @@ -366,6 +363,9 @@ virJSONValuePtr virNetServerServicePreExecRestart(virNetServerServicePtr svc) if (virJSONValueObjectAppendNumberUint(object, "nrequests_client_max", svc->nrequests_client_max) < 0) goto error; + if (!(socks = virJSONValueNewArray())) + goto error; + if (virJSONValueObjectAppend(object, "socks", socks) < 0) { virJSONValueFree(socks); goto error; -- GitLab