[ovs-dev] [leaks 3/4] ofproto-dpif: Fix another memory leak in type_run().
Ben Pfaff
blp at nicira.com
Tue Dec 18 21:51:04 UTC 2012
'devname' needs to be destroyed whether it's the datapath device or not.
Signed-off-by: Ben Pfaff <blp at nicira.com>
---
ofproto/ofproto-dpif.c | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c
index b78d6cf..6ce9a07 100644
--- a/ofproto/ofproto-dpif.c
+++ b/ofproto/ofproto-dpif.c
@@ -836,7 +836,7 @@ type_run(const char *type)
/* Don't report on the datapath's device. */
if (!strcmp(devname, dpif_base_name(backer->dpif))) {
- continue;
+ goto next;
}
ofproto = lookup_ofproto_dpif_by_port_name(devname);
@@ -857,6 +857,7 @@ type_run(const char *type)
}
dpif_port_destroy(&port);
+ next:
free(devname);
}
--
1.7.2.5
More information about the dev
mailing list