util: Remove return value from virTypedParamsCopy

It is already nonfallible, so just change the return type to void.

Signed-off-by: Martin Kletzander <mkletzan@redhat.com>
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

diff --git a/src/libvirt-domain.c b/src/libvirt-domain.c
index 1c2f1ce71c16958efef7f2be559f0deb7354397a..56c2abf3fae72a4e607f0c260732cb78704db8ba 100644 (file)
--- a/src/libvirt-domain.c
+++ b/src/libvirt-domain.c
@@ -3258,8 +3258,7 @@ virDomainMigrateVersion3Full(virDomainPtr domain,
         return NULL;
     }
 
-    if (virTypedParamsCopy(&tmp, params, nparams) < 0)
-        return NULL;
+    virTypedParamsCopy(&tmp, params, nparams);
     params = tmp;
 
     ret = VIR_DRV_SUPPORTS_FEATURE(domain->conn->driver, domain->conn,

diff --git a/src/util/viridentity.c b/src/util/viridentity.c
index 46dcf3a04e4f7727fe12619a024567a29a90b27d..4e9a783cee2ce16cd1f702be09e67bb0413e21e4 100644 (file)
--- a/src/util/viridentity.c
+++ b/src/util/viridentity.c
@@ -420,8 +420,7 @@ virIdentity *virIdentityNewCopy(virIdentity *src)
 {
     g_autoptr(virIdentity) ident = virIdentityNew();
 
-    if (virTypedParamsCopy(&ident->params, src->params, src->nparams) < 0)
-        return NULL;
+    virTypedParamsCopy(&ident->params, src->params, src->nparams);
     ident->nparams = src->nparams;
     ident->maxparams = src->nparams;
 
@@ -826,8 +825,8 @@ int virIdentitySetParameters(virIdentity *ident,
     ident->params = NULL;
     ident->nparams = 0;
     ident->maxparams = 0;
-    if (virTypedParamsCopy(&ident->params, params, nparams) < 0)
-        return -1;
+
+    virTypedParamsCopy(&ident->params, params, nparams);
     ident->nparams = nparams;
     ident->maxparams = nparams;
 
@@ -839,8 +838,7 @@ virTypedParamList *virIdentityGetParameters(virIdentity *ident)
 {
     virTypedParameter *tmp = NULL;
 
-    if (virTypedParamsCopy(&tmp, ident->params, ident->nparams) < 0)
-        return NULL;
+    virTypedParamsCopy(&tmp, ident->params, ident->nparams);
 
     return virTypedParamListFromParams(&tmp, ident->nparams);
 }

diff --git a/src/util/virtypedparam.c b/src/util/virtypedparam.c
index 2d7e4ab35470f0f99485a65b01daf78505fa8d5f..ae676d02fe931adc3b4ca51c3db077eb055475d5 100644 (file)
--- a/src/util/virtypedparam.c
+++ b/src/util/virtypedparam.c
@@ -347,7 +347,7 @@ virTypedParamsReplaceString(virTypedParameterPtr *params,
 }
 
 
-int
+void
 virTypedParamsCopy(virTypedParameterPtr *dst,
                    virTypedParameterPtr src,
                    int nparams)
@@ -356,7 +356,7 @@ virTypedParamsCopy(virTypedParameterPtr *dst,
 
     *dst = NULL;
     if (!src || nparams <= 0)
-        return 0;
+        return;
 
     *dst = g_new0(virTypedParameter, nparams);
 
@@ -369,8 +369,6 @@ virTypedParamsCopy(virTypedParameterPtr *dst,
             (*dst)[i].value = src[i].value;
         }
     }
-
-    return 0;
 }
 
 

diff --git a/src/util/virtypedparam.h b/src/util/virtypedparam.h
index c4bc58ee8f6aeb8abb74824c7ad15f691fe25f96..936dac24eabca1e8be0c7a10fe27d4f82b33c54f 100644 (file)
--- a/src/util/virtypedparam.h
+++ b/src/util/virtypedparam.h
@@ -86,9 +86,9 @@ int virTypedParamsReplaceString(virTypedParameterPtr *params,
                                 const char *name,
                                 const char *value);
 
-int virTypedParamsCopy(virTypedParameterPtr *dst,
-                       virTypedParameterPtr src,
-                       int nparams);
+void virTypedParamsCopy(virTypedParameterPtr *dst,
+                        virTypedParameterPtr src,
+                        int nparams);
 
 char *virTypedParameterToString(virTypedParameterPtr param);
 

diff --git a/src/vz/vz_driver.c b/src/vz/vz_driver.c
index 4f5e340d536c69d88049aa59205170c7e5606524..d5147a6c0d9c403f38b9a12c5496df230c51869a 100644 (file)
--- a/src/vz/vz_driver.c
+++ b/src/vz/vz_driver.c
@@ -3144,8 +3144,7 @@ vzDomainMigratePerformP2P(virDomainObj *dom,
     int ret = -1;
     int maxparams = nparams;
 
-    if (virTypedParamsCopy(&params, orig_params, nparams) < 0)
-        return -1;
+    virTypedParamsCopy(&params, orig_params, nparams);
 
     if (!(dconn = virConnectOpen(dconnuri)))
         goto done;