diff mbox

[ovs-dev,1/2] ovs-router: fix memory leak reported by valgrind.

Message ID 1486536071-14280-1-git-send-email-u9012063@gmail.com
State Superseded
Headers show

Commit Message

William Tu Feb. 8, 2017, 6:41 a.m. UTC 
Valgrind testcase 772: appctl - route/lookup6 reports the
following definitely lost:
  xmalloc (util.c:112)
  miniflow_alloc (flow.c:2500)
  minimatch_init (match.c:1387)
  rt_entry_delete (ovs-router.c:274)
  ovs_router_del (ovs-router.c:384)
  process_command (unixctl.c:313)

Signed-off-by: William Tu <u9012063@gmail.com>
---
 lib/ovs-router.c | 1 +
 1 file changed, 1 insertion(+)
diff mbox

Patch

diff --git a/lib/ovs-router.c b/lib/ovs-router.c
index d98de20..53fb834 100644
--- a/lib/ovs-router.c
+++ b/lib/ovs-router.c
@@ -280,6 +280,7 @@  rt_entry_delete(uint32_t mark, uint8_t priority,
         res = __rt_entry_delete(cr);
         ovs_mutex_unlock(&mutex);
     }
+    minimatch_destroy(CONST_CAST(struct minimatch *, &rule.match));
     return res;
 }