On Thu, Dec 17, 2015 at 5:28 PM, Cheng-Chun Tu <u9012...@gmail.com> wrote: > Hi Ben, > > I've applied your patch and valgrind no longer complain about > tun_metadata_add_entry. However, another issue found (reported by testcase > 643), patch below provided the fix and testcase 643 passes. > > valgrind message > ==20490== by 0x4E1384: xmalloc (util.c:112) > ==20490== by 0x468CD1: resize (hmap.c:100) > ==20490== by 0x4DE87B: hmap_insert_at (hmap.h:235) > ==20490== by 0x4DE87B: table_alloc (tun-metadata.c:106) > ==20490== by 0x4DEF93: tun_metadata_table_mod (tun-metadata.c:164) > ==20490== by 0x420D85: handle_geneve_table_mod (ofproto.c:7080) > ==20490== by 0x420D85: handle_openflow__ (ofproto.c:7245)
This patch looks right to me, can you submit it formally with a signed-off-by? Thanks for tracking all of these down! _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
