In testcase "ofproto-dpif - clone action", valgrind reports a memory leak
with the following call stack.
    xrealloc (util.c:123)
    ds_reserve (dynamic-string.c:63)
    ds_put_format_valist (dynamic-string.c:161)
    ds_put_format (dynamic-string.c:142)
    disable_datapath_clone (ofproto-dpif.c:5206)
    process_command (unixctl.c:313)
    run_connection (unixctl.c:347)
    unixctl_server_run (unixctl.c:400)
    main (ovs-vswitchd.c:112)

Signed-off-by: Yi-Hung Wei <[email protected]>
---
 ofproto/ofproto-dpif.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-dpif.c
index 9556a0e0cd58..96337676719c 100644
--- a/ofproto/ofproto-dpif.c
+++ b/ofproto/ofproto-dpif.c
@@ -5205,6 +5205,7 @@ disable_datapath_clone(struct unixctl_conn *conn 
OVS_UNUSED,
     udpif_flush(ofproto->backer->udpif);
     ds_put_format(&ds, "Datapath clone action disabled for bridge %s", br);
     unixctl_command_reply(conn, ds_cstr(&ds));
+    ds_destroy(&ds);
 }
 
 static void
-- 
2.7.4

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to