[libvirt] [PATCH] networkGetDHCPLeases: Remove unnecessary error reporting

Nehal J Wani nehaljw.kkd1 at gmail.com
Mon Jan 5 17:49:24 UTC 2015


Lack of a lease (whether mac is given or not) is a normal expected scenario.
There is no need to raise an error.

Signed-off-by: Nehal J Wani <nehaljw.kkd1 at gmail.com>
---
 src/network/bridge_driver.c | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/src/network/bridge_driver.c b/src/network/bridge_driver.c
index 29222d0..6a49a23 100644
--- a/src/network/bridge_driver.c
+++ b/src/network/bridge_driver.c
@@ -3654,12 +3654,6 @@ networkGetDHCPLeases(virNetworkPtr network,
         VIR_FREE(lease);
     }
 
-    if (need_results && mac && !leases_ret) {
-        virReportError(VIR_ERR_INTERNAL_ERROR,
-                       _("no lease with matching MAC address: %s"), mac);
-        goto error;
-    }
-
     if (leases_ret) {
         /* NULL terminated array */
         ignore_value(VIR_REALLOC_N(leases_ret, nleases + 1));
-- 
2.1.0




More information about the libvir-list mailing list