[PATCHv3 02/17] conf: use VIR_(INSERT|DELETE)_ELEMENT in virNetworkUpdate backend

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



The already-written backend functions for virNetworkUpdate that add
and delete items into lists within the a network were already debugged
to work properly, but future such functions will use
VIR_(INSERT|DELETE)_ELEMENT instead, so these are changed for
uniformity.
---
 src/conf/network_conf.c | 91 ++++++++++++++-----------------------------------
 src/conf/network_conf.h |  4 +--
 2 files changed, 27 insertions(+), 68 deletions(-)

diff --git a/src/conf/network_conf.c b/src/conf/network_conf.c
index 7220659..777b89f 100644
--- a/src/conf/network_conf.c
+++ b/src/conf/network_conf.c
@@ -2495,27 +2495,15 @@ virNetworkDefUpdateIPDHCPHost(virNetworkDefPtr def,
                 goto cleanup;
             }
         }
+
         /* add to beginning/end of list */
-        if (VIR_REALLOC_N(ipdef->hosts, ipdef->nhosts +1) < 0) {
+        if (VIR_INSERT_ELEMENT(ipdef->hosts,
+                               command == VIR_NETWORK_UPDATE_COMMAND_ADD_FIRST
+                               ? 0 : ipdef->nhosts,
+                               ipdef->nhosts, &host) < 0) {
             virReportOOMError();
             goto cleanup;
         }
-
-        if (command == VIR_NETWORK_UPDATE_COMMAND_ADD_LAST) {
-
-            ipdef->hosts[ipdef->nhosts] = host;
-            ipdef->nhosts++;
-            memset(&host, 0, sizeof(host));
-
-        } else { /* implied (command == VIR_NETWORK_UPDATE_COMMAND_ADD_FIRST) */
-
-            memmove(ipdef->hosts + 1, ipdef->hosts,
-                    sizeof(*ipdef->hosts) * ipdef->nhosts);
-            ipdef->hosts[0] = host;
-            ipdef->nhosts++;
-            memset(&host, 0, sizeof(host));
-        }
-
     } else if (command == VIR_NETWORK_UPDATE_COMMAND_DELETE) {
 
         if (virNetworkDHCPHostDefParse(def->name, ctxt->node, &host, false) < 0)
@@ -2541,10 +2529,8 @@ virNetworkDefUpdateIPDHCPHost(virNetworkDefPtr def,
 
         /* remove it */
         virNetworkDHCPHostDefClear(&ipdef->hosts[ii]);
-        memmove(ipdef->hosts + ii, ipdef->hosts + ii + 1,
-                sizeof(*ipdef->hosts) * (ipdef->nhosts - ii - 1));
-        ipdef->nhosts--;
-        ignore_value(VIR_REALLOC_N(ipdef->hosts, ipdef->nhosts));
+        VIR_DELETE_ELEMENT(ipdef->hosts, ii, ipdef->nhosts);
+
     } else {
         virNetworkDefUpdateUnknownCommand(command);
         goto cleanup;
@@ -2615,21 +2601,14 @@ virNetworkDefUpdateIPDHCPRange(virNetworkDefPtr def,
         }
 
         /* add to beginning/end of list */
-        if (VIR_REALLOC_N(ipdef->ranges, ipdef->nranges +1) < 0) {
+        if (VIR_INSERT_ELEMENT(ipdef->ranges,
+                               command == VIR_NETWORK_UPDATE_COMMAND_ADD_FIRST
+                               ? 0 : ipdef->nranges,
+                               ipdef->nranges, &range) < 0) {
             virReportOOMError();
             goto cleanup;
         }
 
-        if (command == VIR_NETWORK_UPDATE_COMMAND_ADD_LAST) {
-            ipdef->ranges[ipdef->nranges] = range;
-        } else { /* implied (command == VIR_NETWORK_UPDATE_COMMAND_ADD_FIRST) */
-            memmove(ipdef->ranges + 1, ipdef->ranges,
-                    sizeof(*ipdef->ranges) * ipdef->nranges);
-            ipdef->ranges[0] = range;
-        }
-        ipdef->nranges++;
-        memset(&range, 0, sizeof(range));
-
     } else if (command == VIR_NETWORK_UPDATE_COMMAND_DELETE) {
 
         if (ii == ipdef->nranges) {
@@ -2641,10 +2620,8 @@ virNetworkDefUpdateIPDHCPRange(virNetworkDefPtr def,
 
         /* remove it */
         /* NB: nothing to clear from a RangeDef that's being freed */
-        memmove(ipdef->ranges + ii, ipdef->ranges + ii + 1,
-                sizeof(*ipdef->ranges) * (ipdef->nranges - ii - 1));
-        ipdef->nranges--;
-        ignore_value(VIR_REALLOC_N(ipdef->ranges, ipdef->nranges));
+        VIR_DELETE_ELEMENT(ipdef->ranges, ii, ipdef->nranges);
+
     } else {
         virNetworkDefUpdateUnknownCommand(command);
         goto cleanup;
@@ -2719,21 +2696,14 @@ virNetworkDefUpdateForwardInterface(virNetworkDefPtr def,
         }
 
         /* add to beginning/end of list */
-        if (VIR_REALLOC_N(def->forwardIfs, def->nForwardIfs + 1) < 0) {
+        if (VIR_INSERT_ELEMENT(def->forwardIfs,
+                               command == VIR_NETWORK_UPDATE_COMMAND_ADD_FIRST
+                               ? 0 : def->nForwardIfs,
+                               def->nForwardIfs, &iface) < 0) {
             virReportOOMError();
             goto cleanup;
         }
 
-        if (command == VIR_NETWORK_UPDATE_COMMAND_ADD_LAST) {
-            def->forwardIfs[def->nForwardIfs] = iface;
-        } else { /* implied (command == VIR_NETWORK_UPDATE_COMMAND_ADD_FIRST) */
-            memmove(def->forwardIfs + 1, def->forwardIfs,
-                    sizeof(*def->forwardIfs) * def->nForwardIfs);
-            def->forwardIfs[0] = iface;
-        }
-        def->nForwardIfs++;
-        memset(&iface, 0, sizeof(iface));
-
     } else if (command == VIR_NETWORK_UPDATE_COMMAND_DELETE) {
 
         if (ii == def->nForwardIfs) {
@@ -2757,10 +2727,8 @@ virNetworkDefUpdateForwardInterface(virNetworkDefPtr def,
 
         /* remove it */
         virNetworkForwardIfDefClear(&def->forwardIfs[ii]);
-        memmove(def->forwardIfs + ii, def->forwardIfs + ii + 1,
-                sizeof(*def->forwardIfs) * (def->nForwardIfs - ii - 1));
-        def->nForwardIfs--;
-        ignore_value(VIR_REALLOC_N(def->forwardIfs, def->nForwardIfs));
+        VIR_DELETE_ELEMENT(def->forwardIfs, ii, def->nForwardIfs);
+
     } else {
         virNetworkDefUpdateUnknownCommand(command);
         goto cleanup;
@@ -2855,29 +2823,20 @@ virNetworkDefUpdatePortGroup(virNetworkDefPtr def,
         (command == VIR_NETWORK_UPDATE_COMMAND_ADD_LAST)) {
 
         /* add to beginning/end of list */
-        if (VIR_REALLOC_N(def->portGroups, def->nPortGroups +1) < 0) {
+        if (VIR_INSERT_ELEMENT(def->portGroups,
+                               command == VIR_NETWORK_UPDATE_COMMAND_ADD_FIRST
+                               ? 0 : def->nPortGroups,
+                               def->nPortGroups, &portgroup) < 0) {
             virReportOOMError();
             goto cleanup;
         }
 
-        if (command == VIR_NETWORK_UPDATE_COMMAND_ADD_LAST) {
-            def->portGroups[def->nPortGroups] = portgroup;
-        } else { /* implied (command == VIR_NETWORK_UPDATE_COMMAND_ADD_FIRST) */
-            memmove(def->portGroups + 1, def->portGroups,
-                    sizeof(*def->portGroups) * def->nPortGroups);
-            def->portGroups[0] = portgroup;
-        }
-        def->nPortGroups++;
-        memset(&portgroup, 0, sizeof(portgroup));
-
     } else if (command == VIR_NETWORK_UPDATE_COMMAND_DELETE) {
 
         /* remove it */
         virPortGroupDefClear(&def->portGroups[foundName]);
-        memmove(def->portGroups + foundName, def->portGroups + foundName + 1,
-                sizeof(*def->portGroups) * (def->nPortGroups - foundName - 1));
-        def->nPortGroups--;
-        ignore_value(VIR_REALLOC_N(def->portGroups, def->nPortGroups));
+        VIR_DELETE_ELEMENT(def->portGroups, foundName, def->nPortGroups);
+
     } else {
         virNetworkDefUpdateUnknownCommand(command);
         goto cleanup;
diff --git a/src/conf/network_conf.h b/src/conf/network_conf.h
index 949b3d2..87d7746 100644
--- a/src/conf/network_conf.h
+++ b/src/conf/network_conf.h
@@ -129,10 +129,10 @@ struct _virNetworkIpDef {
     unsigned int prefix;        /* ipv6 - only prefix allowed */
     virSocketAddr netmask;      /* ipv4 - either netmask or prefix specified */
 
-    unsigned int nranges;        /* Zero or more dhcp ranges */
+    size_t nranges;             /* Zero or more dhcp ranges */
     virNetworkDHCPRangeDefPtr ranges;
 
-    unsigned int nhosts;         /* Zero or more dhcp hosts */
+    size_t nhosts;              /* Zero or more dhcp hosts */
     virNetworkDHCPHostDefPtr hosts;
 
     char *tftproot;
-- 
1.7.11.7

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]