From c83b89ab6da8704b140ec51cd5bae10448b510cb Mon Sep 17 00:00:00 2001 From: Ben Pfaff Date: Tue, 18 Dec 2012 13:18:26 -0800 Subject: [PATCH] ofproto-dpif: Fix another memory leak in type_run(). 'devname' needs to be destroyed whether it's the datapath device or not. Signed-off-by: Ben Pfaff --- ofproto/ofproto-dpif.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c index b78d6cf55..6ce9a07c4 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); } -- 2.43.0