Re: [ovs-dev] [PATCHv2] ovn-nbctl: Fix memory leak reported by Valgrind.

2016-05-15 Thread William Tu
yes, using bool is better. Resubmit patch here: https://patchwork.ozlabs.org/patch/622407/ Thanks, William On Sat, May 14, 2016 at 9:48 PM, Ben Pfaff wrote: > On Sat, May 14, 2016 at 04:08:42PM -0700, William Tu wrote: > > Definitely lost is reported by test 2026: ovn -- 3 HVs, 1 LS, 3 > lports

Re: [ovs-dev] [PATCHv2] ovn-nbctl: Fix memory leak reported by Valgrind.

2016-05-14 Thread Ben Pfaff
On Sat, May 14, 2016 at 04:08:42PM -0700, William Tu wrote: > Definitely lost is reported by test 2026: ovn -- 3 HVs, 1 LS, 3 lports/HV. > ds_put_char__ (dynamic-string.c:82) > ds_put_char (dynamic-string.h:88) > process_escape_args (process.c:103) > main (ovn-nbctl.c:92) > Another leak sho

[ovs-dev] [PATCHv2] ovn-nbctl: Fix memory leak reported by Valgrind.

2016-05-14 Thread William Tu
Definitely lost is reported by test 2026: ovn -- 3 HVs, 1 LS, 3 lports/HV. ds_put_char__ (dynamic-string.c:82) ds_put_char (dynamic-string.h:88) process_escape_args (process.c:103) main (ovn-nbctl.c:92) Another leak shown at ovn-sbctl.c with similar pattern. Signed-off-by: William Tu ---