Valgrind complains in test 2359 (): 864 (576 direct, 288 indirect) bytes in 18 blocks are definitely lost in loss record 96 of 101 by 0x4A6D64: xmalloc (util.c:120) by 0x40BC04: gateway_chassis_get_ordered (gchassis.c:73) by 0x408CF0: bfd_calculate_chassis (bfd.c:219) by 0x408CF0: bfd_run (bfd.c:257) by 0x407F72: main (ovn-controller.c:718)
gateway_chassis wasn't released before the 'continue' line. Signed-off-by: Yifeng Sun <[email protected]> --- ovn/controller/bfd.c | 1 + 1 file changed, 1 insertion(+) diff --git a/ovn/controller/bfd.c b/ovn/controller/bfd.c index efc1482ecaa2..8f020d597083 100644 --- a/ovn/controller/bfd.c +++ b/ovn/controller/bfd.c @@ -221,6 +221,7 @@ bfd_calculate_chassis(struct controller_ctx *ctx, /* we don't need BFD for non-HA chassisredirect */ if (!gateway_chassis || ovs_list_is_short(gateway_chassis)) { + gateway_chassis_destroy(gateway_chassis); continue; } our_chassis_is_gw_for_dp = gateway_chassis_contains( -- 2.7.4 _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
