On Fri, Aug 19, 2016 at 9:06 PM, Ramu Ramamurthy <ramu.ramamur...@gmail.com> wrote:
> From: Ramu Ramamurthy <ramu.ramamur...@us.ibm.com> > > The following leaks are due to missing ds_destroy in a few > places in build_acl. > > 5,850 bytes in 50 blocks are definitely lost in loss record 93 of 93 > at 0x4C29BFD: malloc (in /usr/lib64/valgrind/vgpreload_ > memcheck-amd64-linux.so) > by 0x4C2BACB: realloc (in /usr/lib64/valgrind/vgpreload_ > memcheck-amd64-linux.so) > by 0x449507: xrealloc (util.c:123) > by 0x42CC73: ds_reserve (dynamic-string.c:63) > by 0x42D08F: ds_put_format_valist (dynamic-string.c:161) > by 0x42D176: ds_put_format (dynamic-string.c:142) > by 0x40D380: build_acls (ovn-northd.c:2320) > by 0x40D380: build_lswitch_flows.constprop.36 (ovn-northd.c:2472) > by 0x4072D9: build_lflows (ovn-northd.c:3845) > by 0x4072D9: ovnnb_db_run (ovn-northd.c:3971) > by 0x4072D9: main (ovn-northd.c:4375) > > 9,360 bytes in 72 blocks are definitely lost in loss record 93 of 93 > at 0x4C29BFD: malloc (in /usr/lib64/valgrind/vgpreload_ > memcheck-amd64-linux.so) > by 0x4C2BACB: realloc (in /usr/lib64/valgrind/vgpreload_ > memcheck-amd64-linux.so) > by 0x449507: xrealloc (util.c:123) > by 0x42CC73: ds_reserve (dynamic-string.c:63) > by 0x42D08F: ds_put_format_valist (dynamic-string.c:161) > by 0x42D176: ds_put_format (dynamic-string.c:142) > by 0x40D505: build_acls (ovn-northd.c:2346) > by 0x40D505: build_lswitch_flows.constprop.36 (ovn-northd.c:2472) > by 0x4072D9: build_lflows (ovn-northd.c:3845) > by 0x4072D9: ovnnb_db_run (ovn-northd.c:3971) > by 0x4072D9: main (ovn-northd.c:4375) > --- > ovn/northd/ovn-northd.c | 2 ++ > 1 file changed, 2 insertions(+) > I was about to apply this, but it's missing your Signed-off-by. Can you please re-submit with Signed-off-by included? Thanks, -- Russell Bryant _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev