[ovs-dev] [PATCH v2] ovsdb-client: fix memory leak to prevent valgrind reporting memory leaks while running test suite

[Ben Pfaff]

On Wed, Oct 02, 2019 at 11:37:52AM +0200, Damijan Skvarc wrote:
> memory leaks are reported in several tests and are expressed in a following way:
> 
> ==29840== 208 (48 direct, 160 indirect) bytes in 1 blocks are definitely lost in
>  loss record 43 of 44
> ==29840==    at 0x4C2FB55: calloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64
> -linux.so)
> ==29840==    by 0x449D12: xcalloc (util.c:121)
> ==29840==    by 0x432949: jsonrpc_msg_from_json (jsonrpc.c:697)
> ==29840==    by 0x432A8F: jsonrpc_parse_received_message (jsonrpc.c:472)
> ==29840==    by 0x432A8F: jsonrpc_recv.part.7 (jsonrpc.c:338)
> ==29840==    by 0x4338F7: jsonrpc_recv (jsonrpc.c:1139)
> ==29840==    by 0x4338F7: jsonrpc_session_recv (jsonrpc.c:1112)
> ==29840==    by 0x40719B: do_wait (ovsdb-client.c:2463)
> ==29840==    by 0x405F76: main (ovsdb-client.c:282)
> 
> Signed-off-by: Damijan Skvarc <damjan.skvarc at gmail.com>

Thanks.  I applied this to master.

I changed this exit(0) to exit(EXIT_FAILURE), since that's what
ovs_fatal() uses:

> -                    ovs_fatal(0, "%s: set_db_change_aware failed (%s)",
> +                    ovs_error(0, "%s: set_db_change_aware failed (%s)",
>                                jsonrpc_session_get_name(js),
>                                json_to_string(reply->error, 0));
> +                    jsonrpc_msg_destroy(reply);
> +                    exit(0);