util: virtypedparam: Introduce virTypedParamListNew()

Add an allocator function and refactor all allocations to use it. In
upcoming patches 'struct _virTypedParamList' will be made private.

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

diff --git a/src/admin/admin_server.c b/src/admin/admin_server.c
index 7ed2645f57aa2f0d7bda0b20a5582cb992a0260d..394de0dfab66b0fc8bca15270628cc738bf631df 100644 (file)
--- a/src/admin/admin_server.c
+++ b/src/admin/admin_server.c
@@ -76,7 +76,7 @@ adminServerGetThreadPoolParameters(virNetServer *srv,
     size_t freeWorkers;
     size_t nPrioWorkers;
     size_t jobQueueDepth;
-    g_autoptr(virTypedParamList) paramlist = g_new0(virTypedParamList, 1);
+    g_autoptr(virTypedParamList) paramlist = virTypedParamListNew();
 
     virCheckFlags(0, -1);
 
@@ -200,7 +200,7 @@ adminClientGetInfo(virNetServerClient *client,
     bool readonly;
     g_autofree char *sock_addr = NULL;
     const char *attr = NULL;
-    g_autoptr(virTypedParamList) paramlist = g_new0(virTypedParamList, 1);
+    g_autoptr(virTypedParamList) paramlist = virTypedParamListNew();
     g_autoptr(virIdentity) identity = NULL;
     int rc;
 
@@ -298,7 +298,7 @@ adminServerGetClientLimits(virNetServer *srv,
                            int *nparams,
                            unsigned int flags)
 {
-    g_autoptr(virTypedParamList) paramlist = g_new0(virTypedParamList, 1);
+    g_autoptr(virTypedParamList) paramlist = virTypedParamListNew();
 
     virCheckFlags(0, -1);
 

diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms
index 1247b67a3953265dc451d3b070c68e1ec32808dd..c596ef0f87282166b39b77f02163434a6c4b6a3c 100644 (file)
--- a/src/libvirt_private.syms
+++ b/src/libvirt_private.syms
@@ -3557,6 +3557,7 @@ virTypedParamListAddUInt;
 virTypedParamListAddULLong;
 virTypedParamListFree;
 virTypedParamListFromParams;
+virTypedParamListNew;
 virTypedParamListStealParams;
 virTypedParamsCheck;
 virTypedParamsCopy;

diff --git a/src/qemu/qemu_domainjob.c b/src/qemu/qemu_domainjob.c
index 6376f928d5f3ee53c64f1e7543783230836151a6..4df1af0ce753186da637a76bcccad6bdd990a371 100644 (file)
--- a/src/qemu/qemu_domainjob.c
+++ b/src/qemu/qemu_domainjob.c
@@ -478,7 +478,7 @@ qemuDomainBackupJobDataToParams(virDomainJobData *jobData,
 {
     qemuDomainJobDataPrivate *priv = jobData->privateData;
     qemuDomainBackupStats *stats = &priv->stats.backup;
-    g_autoptr(virTypedParamList) par = g_new0(virTypedParamList, 1);
+    g_autoptr(virTypedParamList) par = virTypedParamListNew();
 
     if (virTypedParamListAddInt(par, jobData->operation,
                                 VIR_DOMAIN_JOB_OPERATION) < 0)

diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index 143419b4fcb5f6888375ba04c82c8b50ee528330..a2b24babacca32cd9d56d2d557644748cc922835 100644 (file)
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@ -17934,7 +17934,7 @@ qemuDomainGetStats(virConnectPtr conn,
     g_autoptr(virTypedParamList) params = NULL;
     size_t i;
 
-    params = g_new0(virTypedParamList, 1);
+    params = virTypedParamListNew();
 
     for (i = 0; qemuDomainGetStatsWorkers[i].func; i++) {
         if (stats & qemuDomainGetStatsWorkers[i].stats) {

diff --git a/src/test/test_driver.c b/src/test/test_driver.c
index 7b06896d441efc7e09438996a1dad0619fb7ef2a..47c74c420c9844c3d53997a14c1ab8ed2d36854d 100644 (file)
--- a/src/test/test_driver.c
+++ b/src/test/test_driver.c
@@ -9858,7 +9858,7 @@ testDomainGetStats(virConnectPtr conn,
     g_autoptr(virTypedParamList) params = NULL;
     size_t i;
 
-    params = g_new0(virTypedParamList, 1);
+    params = virTypedParamListNew();
 
     for (i = 0; testDomainGetStatsWorkers[i].func; i++) {
         if (stats & testDomainGetStatsWorkers[i].stats) {

diff --git a/src/util/virtypedparam.c b/src/util/virtypedparam.c
index 1a1a34fdb2d921eb0c56e9a4d09db0342f18b5d4..de3a4e76b4f56ae9472f2e4ced5de40bec03b3d8 100644 (file)
--- a/src/util/virtypedparam.c
+++ b/src/util/virtypedparam.c
@@ -705,6 +705,13 @@ virTypedParamsSerialize(virTypedParameterPtr params,
 }
 
 
+virTypedParamList *
+virTypedParamListNew(void)
+{
+    return g_new0(virTypedParamList, 1);
+}
+
+
 void
 virTypedParamListFree(virTypedParamList *list)
 {
@@ -733,7 +740,7 @@ virTypedParamList *
 virTypedParamListFromParams(virTypedParameterPtr *params,
                             size_t nparams)
 {
-    virTypedParamList *l = g_new0(virTypedParamList, 1);
+    virTypedParamList *l = virTypedParamListNew();
 
     l->par = g_steal_pointer(params);
     l->npar = nparams;

diff --git a/src/util/virtypedparam.h b/src/util/virtypedparam.h
index fb9815c661bed7395522e144b949ff71b7ed61bf..4aa597bc811e7a00f13480e71982030433af7ce2 100644 (file)
--- a/src/util/virtypedparam.h
+++ b/src/util/virtypedparam.h
@@ -146,6 +146,7 @@ struct _virTypedParamList {
 void
 virTypedParamListFree(virTypedParamList *list);
 G_DEFINE_AUTOPTR_CLEANUP_FUNC(virTypedParamList, virTypedParamListFree);
+virTypedParamList *virTypedParamListNew(void);
 
 size_t
 virTypedParamListStealParams(virTypedParamList *list,