On Thu, Feb 26, 2015 at 15:17:15 +0100, Michal Privoznik wrote:
All of our vir*Free() functions should accept NULL, even though
that there's no way of actually passing NULL with current code.
Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com>
---
src/conf/network_conf.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/conf/network_conf.c b/src/conf/network_conf.c
index 154a9bc..9734a7f 100644
--- a/src/conf/network_conf.c
+++ b/src/conf/network_conf.c
@@ -279,6 +279,9 @@ void virNetworkObjListFree(virNetworkObjListPtr nets)
{
size_t i;
+ if (!nets)
+ return;
+
for (i = 0; i < nets->count; i++)
virNetworkObjFree(nets->objs[i]);
Since the function does not free @nets at the end it should really be
called virNetworkObjListClear. But that was present previously too.
ACK as is.
Peter