"dev" <[email protected]> wrote on 06/05/2016 09:37:35 AM:
> From: William Tu <[email protected]> > To: [email protected] > Date: 06/05/2016 09:37 AM > Subject: [ovs-dev] [PATCH] ovn-controller: Fix memory leak reported > by valgrind. > Sent by: "dev" <[email protected]> > > Calling ovsdb_idl_set_remote() might overwrite the 'idl->session'. The patch > fixes them by freeing 'idl->session' before it is overwritten. > > Testcast ovn-controller - ovn-bridge-mappings reports two definitelylosts in: > xmalloc (util.c:112) > jsonrpc_session_open (jsonrpc.c:784) > ovsdb_idl_create (ovsdb-idl.c:246) > main (ovn-controller.c:384) > and, > xmalloc (util.c:112) > jsonrpc_session_open (jsonrpc.c:784) > ovsdb_idl_set_remote (ovsdb-idl.c:289) > main (ovn-controller.c:409) > > Signed-off-by: William Tu <[email protected]> Looked at this by inspection and it makes sense. I also checked ovsdb/jsonrpc-server.c but I don't think that usage leaks - William, did you also check that? Acked-by: Ryan Moats <[email protected]> _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
