From: Mohammad Heib <[email protected]>

unixctl_client_transact allocate memory using xstrdup and
this memory must be freed before exiting.

Signed-off-by: Mohammad Heib <[email protected]>
---
 utilities/ovn-appctl.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/utilities/ovn-appctl.c b/utilities/ovn-appctl.c
index 912e96890..b90879346 100644
--- a/utilities/ovn-appctl.c
+++ b/utilities/ovn-appctl.c
@@ -68,7 +68,8 @@ main(int argc, char *argv[])
         jsonrpc_close(client);
         fputs(cmd_error, stderr);
         ovs_error(0, "%s: server returned an error", target);
-        exit(2);
+        free(cmd_error);
+        error? exit(error) : exit(2);
     } else if (cmd_result) {
         fputs(cmd_result, stdout);
     } else {
-- 
2.27.0

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

Reply via email to