Re: [ovs-dev] [PATCH 3/5] ovs-vsctl, vtep-ctl: Free 'args' string on exit.

2018-01-26 Thread Ben Pfaff
On Fri, Jan 26, 2018 at 07:45:26AM -0800, William Tu wrote: > On Thu, Jan 25, 2018 at 3:39 PM, Ben Pfaff wrote: > > This avoids a memory leak warning from valgrind. > > > > ovn-sbctl and ovn-nbctl already followed this pattern. > > > > Signed-off-by: Ben Pfaff > > ---

Re: [ovs-dev] [PATCH 3/5] ovs-vsctl, vtep-ctl: Free 'args' string on exit.

2018-01-26 Thread William Tu
On Thu, Jan 25, 2018 at 3:39 PM, Ben Pfaff wrote: > This avoids a memory leak warning from valgrind. > > ovn-sbctl and ovn-nbctl already followed this pattern. > > Signed-off-by: Ben Pfaff > --- Looks good to me. Acked-by: William Tu >

[ovs-dev] [PATCH 3/5] ovs-vsctl, vtep-ctl: Free 'args' string on exit.

2018-01-25 Thread Ben Pfaff
This avoids a memory leak warning from valgrind. ovn-sbctl and ovn-nbctl already followed this pattern. Signed-off-by: Ben Pfaff --- utilities/ovs-vsctl.c | 12 vtep/vtep-ctl.c | 12 2 files changed, 16 insertions(+), 8 deletions(-) diff --git