From: Ben Pfaff <blp@nicira.com>
Date: Wed, 26 Dec 2012 17:42:33 +0000 (-0800)
Subject: connmgr: Fix memory leak destroying ofmonitor.
X-Git-Tag: sliver-openvswitch-1.9.90-3~10^2~30
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=5684eb749042325111e05f41b20258e6824fc0b5;p=sliver-openvswitch.git

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 <blp@nicira.com>
Acked-by: Ethan Jackson <ethan@nicira.com>
---

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);
     }