ovsdb-client: Fix memory leak reported by valgrind.
authorWilliam Tu <u9012063@gmail.com>
Thu, 7 Jan 2016 23:53:09 +0000 (15:53 -0800)
committerBen Pfaff <blp@ovn.org>
Mon, 11 Jan 2016 17:07:24 +0000 (09:07 -0800)
testc ase 1427: ovsdb-server combines updates on backlogged connections.
valgrind reports two leaks:
    unixctl_server_create (unixctl.c:250)
    do_monitor__ (ovsdb-client.c:918)
and
    json_create (json.c:1406)
    json_integer_create (json.c:262)
    json_clone (json.c:413)
    do_monitor__ (ovsdb-client.c:958)

Signed-off-by: William Tu <u9012063@gmail.com>
Acked-by: Andy Zhou <azhou@ovn.org>
Signed-off-by: Ben Pfaff <blp@ovn.org>
ovsdb/ovsdb-client.c

index 7a33ef8..06155ec 100644 (file)
@@ -1037,6 +1037,9 @@ do_monitor__(struct jsonrpc *rpc, const char *database,
         unixctl_server_wait(unixctl);
         poll_block();
     }
+
+    json_destroy(request_id);
+    unixctl_server_destroy(unixctl);
     ovsdb_schema_destroy(schema);
     destroy_monitored_table(mts, n_mts);
 }