[ovs-dev,1/2] ovs-router: add missing ds_destroy after ds was put
diff mbox

Message ID 1441388408-1273-1-git-send-email-cascardo@redhat.com
State Accepted
Headers show

Commit Message

Thadeu Lima de Souza Cascardo Sept. 4, 2015, 5:40 p.m. UTC
Signed-off-by: Thadeu Lima de Souza Cascardo <cascardo@redhat.com>
---
 lib/ovs-router.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Ben Pfaff Sept. 17, 2015, 9:43 p.m. UTC | #1
On Fri, Sep 04, 2015 at 02:40:07PM -0300, Thadeu Lima de Souza Cascardo wrote:
> Signed-off-by: Thadeu Lima de Souza Cascardo <cascardo@redhat.com>

Thanks, applied to master.

Patch
diff mbox

diff --git a/lib/ovs-router.c b/lib/ovs-router.c
index df55bb4..d4a5c35 100644
--- a/lib/ovs-router.c
+++ b/lib/ovs-router.c
@@ -278,6 +278,7 @@  ovs_router_lookup_cmd(struct unixctl_conn *conn, int argc OVS_UNUSED,
             ds_put_format(&ds, "gateway " IP_FMT "\n", IP_ARGS(gw));
             ds_put_format(&ds, "dev %s\n", iface);
             unixctl_command_reply(conn, ds_cstr(&ds));
+            ds_destroy(&ds);
         } else {
             unixctl_command_reply(conn, "Not found");
         }