From 31d21197d37ca67764e16f51eba886275a4d3ab8 Mon Sep 17 00:00:00 2001 From: Laine Stump Date: Sun, 11 Nov 2012 10:27:55 -0500 Subject: [PATCH] conf: make virNetworkIpDefClear consistent with other functions The other clear functions in network_conf.c that clear out arrays of sub-objects do so by using the n[itemname]s value as a counter going down to 0. Make this one consistent. There's no functional value, just makes the style more consistent with the rest of the file. --- src/conf/network_conf.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/conf/network_conf.c b/src/conf/network_conf.c index 0091c8a7c2..9213371b80 100644 --- a/src/conf/network_conf.c +++ b/src/conf/network_conf.c @@ -118,15 +118,14 @@ virNetworkDHCPHostDefClear(virNetworkDHCPHostDefPtr def) VIR_FREE(def->name); } -static void virNetworkIpDefClear(virNetworkIpDefPtr def) +static void +virNetworkIpDefClear(virNetworkIpDefPtr def) { - int ii; - VIR_FREE(def->family); VIR_FREE(def->ranges); - for (ii = 0 ; ii < def->nhosts && def->hosts ; ii++) - virNetworkDHCPHostDefClear(&def->hosts[ii]); + while (def->nhosts--) + virNetworkDHCPHostDefClear(&def->hosts[def->nhosts]); VIR_FREE(def->hosts); VIR_FREE(def->tftproot); -- GitLab