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>
ofmonitor_destroy(struct ofmonitor *m)
{
if (m) {
+ minimatch_destroy(&m->match);
hmap_remove(&m->ofconn->monitors, &m->ofconn_node);
free(m);
}