On Tue, Oct 19, 2021 at 8:20 AM <[email protected]> wrote:
>
> 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]>

Thanks.  I applied to the main branch.

Numan

> ---
>  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
>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to