remote: use g_strfreev for free()ing lists of strings

Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>

diff --git a/src/remote/remote_daemon_config.c b/src/remote/remote_daemon_config.c
index 2a41c27dfcd659b88bb7857a816a6a52ac041da8..c1e75444e1803ec51ee2da1a3961b44783cc9d53 100644 (file)
--- a/src/remote/remote_daemon_config.c
+++ b/src/remote/remote_daemon_config.c
@@ -168,8 +168,6 @@ daemonConfigNew(bool privileged G_GNUC_UNUSED)
 void
 daemonConfigFree(struct daemonConfig *data)
 {
-    char **tmp;
-
     if (!data)
         return;
 
@@ -179,12 +177,7 @@ daemonConfigFree(struct daemonConfig *data)
     g_free(data->tcp_port);
 #endif /* ! WITH_IP */
 
-    tmp = data->access_drivers;
-    while (tmp && *tmp) {
-        g_free(*tmp);
-        tmp++;
-    }
-    g_free(data->access_drivers);
+    g_strfreev(data->access_drivers);
 
     g_free(data->unix_sock_admin_perms);
     g_free(data->unix_sock_ro_perms);
@@ -192,20 +185,10 @@ daemonConfigFree(struct daemonConfig *data)
     g_free(data->unix_sock_group);
     g_free(data->unix_sock_dir);
 
-    tmp = data->sasl_allowed_username_list;
-    while (tmp && *tmp) {
-        g_free(*tmp);
-        tmp++;
-    }
-    g_free(data->sasl_allowed_username_list);
+    g_strfreev(data->sasl_allowed_username_list);
 
 #ifdef WITH_IP
-    tmp = data->tls_allowed_dn_list;
-    while (tmp && *tmp) {
-        g_free(*tmp);
-        tmp++;
-    }
-    g_free(data->tls_allowed_dn_list);
+    g_strfreev(data->tls_allowed_dn_list);
 
     g_free(data->tls_priority);