From 5684eb749042325111e05f41b20258e6824fc0b5 Mon Sep 17 00:00:00 2001 From: Ben Pfaff Date: Wed, 26 Dec 2012 09:42:33 -0800 Subject: [PATCH] connmgr: Fix memory leak destroying ofmonitor. This should be pretty minor because it only leaks memory if the match does not fit in the inline area of a miniflow and minimask. I imagine that the most common match used for an ofmonitor is "match everything", which wouldn't leak. Signed-off-by: Ben Pfaff Acked-by: Ethan Jackson --- ofproto/connmgr.c | 1 + 1 file changed, 1 insertion(+) diff --git a/ofproto/connmgr.c b/ofproto/connmgr.c index 1dc908263..8a84da5e6 100644 --- a/ofproto/connmgr.c +++ b/ofproto/connmgr.c @@ -1807,6 +1807,7 @@ void ofmonitor_destroy(struct ofmonitor *m) { if (m) { + minimatch_destroy(&m->match); hmap_remove(&m->ofconn->monitors, &m->ofconn_node); free(m); } -- 2.47.0