Super, thanks On Wed, Oct 2, 2019 at 5:31 PM Ben Pfaff <[email protected]> wrote:
> 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 <[email protected]> > > 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); > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
