expr: Fix memory leak reported by valgrind.
authorWilliam Tu <u9012063@gmail.com>
Thu, 11 Feb 2016 17:45:07 +0000 (09:45 -0800)
committerRussell Bryant <russell@ovn.org>
Thu, 11 Feb 2016 20:29:20 +0000 (15:29 -0500)
Testcase 1728: ovn -- 5-term mixed expression normalization
Call stack:
    sset_add__ (sset.c:53)
    crush_and_string (expr.c:1725)
    crush_cmps (expr.c:1998)
    expr_sort (expr.c:2050)
    expr_normalize_and (expr.c:2085)

Signed-off-by: William Tu <u9012063@gmail.com>
Signed-off-by: Daniele Di Proietto <diproiettod@vmware.com>
Co-authored-by: Daniele Di Proietto <diproiettod@vmware.com>
Signed-off-by: Russell Bryant <russell@ovn.org>
ovn/lib/expr.c

index f30500e..316d557 100644 (file)
@@ -1749,6 +1749,7 @@ crush_and_string(struct expr *expr, const struct expr_symbol *symbol)
         sub->cmp.string = xstrdup(string);
         list_push_back(&expr->andor, &sub->node);
     }
+    sset_destroy(&result);
     return expr_fix(expr);
 }