remote: dispatch: Don't use virStringListFreeCount for NULL terminated lists

Both virDomainAuthorizedSSHKeysGet and virDomainGetMessages return a
NULL-terminated string-list, so we can use g_auto(GStrv) to clear the
used memory on failures.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>

diff --git a/src/remote/remote_daemon_dispatch.c b/src/remote/remote_daemon_dispatch.c
index 0e38b314a2ddd04ac4461df839ac993e1ca5a909..0bac7d4f51d1ecd0ed7f5648ecd58b306826fa9a 100644 (file)
--- a/src/remote/remote_daemon_dispatch.c
+++ b/src/remote/remote_daemon_dispatch.c
@@ -7327,7 +7327,7 @@ remoteDispatchDomainAuthorizedSshKeysGet(virNetServer *server G_GNUC_UNUSED,
     int rv = -1;
     virConnectPtr conn = remoteGetHypervisorConn(client);
     int nkeys = 0;
-    char **keys = NULL;
+    g_auto(GStrv) keys = NULL;
     virDomainPtr dom = NULL;
 
     if (!conn)
@@ -7355,8 +7355,6 @@ remoteDispatchDomainAuthorizedSshKeysGet(virNetServer *server G_GNUC_UNUSED,
  cleanup:
     if (rv < 0)
         virNetMessageSaveError(rerr);
-    if (nkeys > 0)
-        virStringListFreeCount(keys, nkeys);
     virObjectUnref(dom);
 
     return rv;
@@ -7409,7 +7407,7 @@ remoteDispatchDomainGetMessages(virNetServer *server G_GNUC_UNUSED,
     int rv = -1;
     virConnectPtr conn = remoteGetHypervisorConn(client);
     int nmsgs = 0;
-    char **msgs = NULL;
+    g_auto(GStrv) msgs = NULL;
     virDomainPtr dom = NULL;
 
     if (!conn)
@@ -7436,8 +7434,6 @@ remoteDispatchDomainGetMessages(virNetServer *server G_GNUC_UNUSED,
  cleanup:
     if (rv < 0)
         virNetMessageSaveError(rerr);
-    if (nmsgs > 0)
-        virStringListFreeCount(msgs, nmsgs);
     virObjectUnref(dom);
 
     return rv;