diff mbox

[ovs-dev,v2] test-ovsdb: fix memory leak reported by valgrind.

Message ID 1453496690-15976-1-git-send-email-u9012063@gmail.com
State Accepted
Headers show

Commit Message

William Tu Jan. 22, 2016, 9:04 p.m. UTC 
Testcase 1311, 1312: Boolean-distinct queries on scalars, reports leak
below:
    xmalloc (util.c:112)
    do_query_distinct (test-ovsdb.c:1208)
    ovs_cmdl_run_command (command-line.c:121)
    main (test-ovsdb.c:72)

Signed-off-by: William Tu <u9012063@gmail.com>
---
 tests/test-ovsdb.c | 2 ++
 1 file changed, 2 insertions(+)

Comments

Ben Pfaff Feb. 6, 2016, 12:08 a.m. UTC | #1 
On Fri, Jan 22, 2016 at 01:04:50PM -0800, William Tu wrote:
> Testcase 1311, 1312: Boolean-distinct queries on scalars, reports leak
> below:
>     xmalloc (util.c:112)
>     do_query_distinct (test-ovsdb.c:1208)
>     ovs_cmdl_run_command (command-line.c:121)
>     main (test-ovsdb.c:72)
> 
> Signed-off-by: William Tu <u9012063@gmail.com>

Applied to master, thanks!
diff mbox

Patch

diff --git a/tests/test-ovsdb.c b/tests/test-ovsdb.c
index 15f41b0..1fbe378 100644
--- a/tests/test-ovsdb.c
+++ b/tests/test-ovsdb.c
@@ -1298,6 +1298,8 @@  do_query_distinct(struct ovs_cmdl_context *ctx)
 
     ovsdb_table_destroy(table); /* Also destroys 'ts'. */
 
+    free(rows);
+    free(classes);
     exit(exit_code);
 }