Re: [ovs-dev] [PATCH] ovn-controller: Fix memory leak reported byvalgrind.

2016-06-09 Thread William Tu
Hi Ryan, Thanks for the review. I looked at the jsonrpc-server.c and I think it's OK without leaks. Regards, William On Wed, Jun 8, 2016 at 7:32 PM, Ryan Moats wrote: > "dev" wrote on 06/05/2016 09:37:35 AM: > >> From: William Tu >> To: dev@openvswitch.org >> Date: 06/05/2016 09:37 AM >> Subj

Re: [ovs-dev] [PATCH] ovn-controller: Fix memory leak reported byvalgrind.

2016-06-08 Thread Ryan Moats
"dev" wrote on 06/05/2016 09:37:35 AM: > From: William Tu > To: dev@openvswitch.org > Date: 06/05/2016 09:37 AM > Subject: [ovs-dev] [PATCH] ovn-controller: Fix memory leak reported > by valgrind. > Sent by: "dev" > > Calling ovsdb_idl_set_remote() might overwrite the 'idl->session'. The patch