[libvirt] [PATCH v1 05/31] virNetworkObjIsDuplicate: s/@doms/@nets/

Michal Privoznik mprivozn at redhat.com
Thu Feb 26 14:17:14 UTC 2015


This is probably a copy-paste error from virDomainObj*
counterpart.  But when speaking of virNetworkObj we should use
variable @nets for an array of networks, rather than @doms. It's
just confusing.

Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
---
 src/conf/network_conf.c | 34 +++++++++++++++++-----------------
 src/conf/network_conf.h |  4 ++--
 2 files changed, 19 insertions(+), 19 deletions(-)

diff --git a/src/conf/network_conf.c b/src/conf/network_conf.c
index 0f4fc1e..154a9bc 100644
--- a/src/conf/network_conf.c
+++ b/src/conf/network_conf.c
@@ -4130,7 +4130,7 @@ virNetworkObjUpdate(virNetworkObjPtr network,
 
 /*
  * virNetworkObjIsDuplicate:
- * @doms : virNetworkObjListPtr to search
+ * @nets : virNetworkObjListPtr to search
  * @def  : virNetworkDefPtr definition of network to lookup
  * @check_active: If true, ensure that network is not active
  *
@@ -4139,32 +4139,32 @@ virNetworkObjUpdate(virNetworkObjPtr network,
  *          1 if network is a duplicate
  */
 int
-virNetworkObjIsDuplicate(virNetworkObjListPtr doms,
+virNetworkObjIsDuplicate(virNetworkObjListPtr nets,
                          virNetworkDefPtr def,
                          bool check_active)
 {
     int ret = -1;
-    virNetworkObjPtr vm = NULL;
+    virNetworkObjPtr net = NULL;
 
-    /* See if a VM with matching UUID already exists */
-    vm = virNetworkFindByUUID(doms, def->uuid);
-    if (vm) {
+    /* See if a network with matching UUID already exists */
+    net = virNetworkFindByUUID(nets, def->uuid);
+    if (net) {
         /* UUID matches, but if names don't match, refuse it */
-        if (STRNEQ(vm->def->name, def->name)) {
+        if (STRNEQ(net->def->name, def->name)) {
             char uuidstr[VIR_UUID_STRING_BUFLEN];
-            virUUIDFormat(vm->def->uuid, uuidstr);
+            virUUIDFormat(net->def->uuid, uuidstr);
             virReportError(VIR_ERR_OPERATION_FAILED,
                            _("network '%s' is already defined with uuid %s"),
-                           vm->def->name, uuidstr);
+                           net->def->name, uuidstr);
             goto cleanup;
         }
 
         if (check_active) {
-            /* UUID & name match, but if VM is already active, refuse it */
-            if (virNetworkObjIsActive(vm)) {
+            /* UUID & name match, but if network is already active, refuse it */
+            if (virNetworkObjIsActive(net)) {
                 virReportError(VIR_ERR_OPERATION_INVALID,
                                _("network is already active as '%s'"),
-                               vm->def->name);
+                               net->def->name);
                 goto cleanup;
             }
         }
@@ -4172,10 +4172,10 @@ virNetworkObjIsDuplicate(virNetworkObjListPtr doms,
         ret = 1;
     } else {
         /* UUID does not match, but if a name matches, refuse it */
-        vm = virNetworkFindByName(doms, def->name);
-        if (vm) {
+        net = virNetworkFindByName(nets, def->name);
+        if (net) {
             char uuidstr[VIR_UUID_STRING_BUFLEN];
-            virUUIDFormat(vm->def->uuid, uuidstr);
+            virUUIDFormat(net->def->uuid, uuidstr);
             virReportError(VIR_ERR_OPERATION_FAILED,
                            _("network '%s' already exists with uuid %s"),
                            def->name, uuidstr);
@@ -4185,8 +4185,8 @@ virNetworkObjIsDuplicate(virNetworkObjListPtr doms,
     }
 
  cleanup:
-    if (vm)
-        virNetworkObjUnlock(vm);
+    if (net)
+        virNetworkObjUnlock(net);
     return ret;
 }
 
diff --git a/src/conf/network_conf.h b/src/conf/network_conf.h
index 484522e..70e6eb5 100644
--- a/src/conf/network_conf.h
+++ b/src/conf/network_conf.h
@@ -309,7 +309,7 @@ virNetworkObjPtr virNetworkFindByName(virNetworkObjListPtr nets,
 
 void virNetworkDefFree(virNetworkDefPtr def);
 void virNetworkObjFree(virNetworkObjPtr net);
-void virNetworkObjListFree(virNetworkObjListPtr vms);
+void virNetworkObjListFree(virNetworkObjListPtr nets);
 
 
 typedef bool (*virNetworkObjListFilter)(virConnectPtr conn,
@@ -413,7 +413,7 @@ virNetworkObjUpdate(virNetworkObjPtr obj,
                     const char *xml,
                     unsigned int flags);  /* virNetworkUpdateFlags */
 
-int virNetworkObjIsDuplicate(virNetworkObjListPtr doms,
+int virNetworkObjIsDuplicate(virNetworkObjListPtr nets,
                              virNetworkDefPtr def,
                              bool check_active);
 
-- 
2.0.5




More information about the libvir-list mailing list