On 1/10/20 2:05 PM, Erik Skultety wrote:
On Thu, Jan 09, 2020 at 01:45:58PM +0100, Michal Privoznik wrote:
From: Julio Faracco <[email protected]>
We have to keep the default - querying the agent if no flag is
set.
Signed-off-by: Michal Privoznik <[email protected]>
Signed-off-by: Julio Faracco <[email protected]>
---
...
+
+static int
+qemuDomainGetHostnameLease(virQEMUDriverPtr driver,
+ virDomainObjPtr vm,
+ char **hostname)
...
+
+ if ((n_leases = virNetworkGetDHCPLeases(network, macaddr,
+ &leases, 0)) < 0)
+ goto endjob;
+
+ for (j = 0; j < n_leases; j++) {
+ virNetworkDHCPLeasePtr lease = leases[j];
+ if (lease->hostname && !*hostname)
+ *hostname = g_strdup(lease->hostname);
Not a big deal, but why not doing a break once you extract the first hostname?
We need to free all leases to avoid memleak. Instead of doing that in
two separate loops (one where we'd look for the hostname, the other just
for freeing), I find it simpler if done in a single loop.
Reviewed-by: Erik Skultety <[email protected]>
Michal
--
libvir-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/libvir-list
