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.
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);