While checking unit tests with valgrind option (make check-valgrind) I have noticed several memory leaks of the following format: ..... ==20019== 13,883 (296 direct, 13,587 indirect) bytes in 1 blocks are definitely lost in loss record 346 of 346 ==20019== at 0x4C2FB55: calloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so) ==20019== by 0x530F52: xcalloc (util.c:121) ==20019== by 0x5037A1: ovsdb_idl_row_create__ (ovsdb-idl.c:3120) ==20019== by 0x5045A3: ovsdb_idl_row_create (ovsdb-idl.c:3133) ==20019== by 0x507240: ovsdb_idl_process_update2 (ovsdb-idl.c:2478) ==20019== by 0x507240: ovsdb_idl_db_parse_update__ (ovsdb-idl.c:2328) ==20019== by 0x507240: ovsdb_idl_db_parse_update (ovsdb-idl.c:2380) ==20019== by 0x508128: ovsdb_idl_process_response (ovsdb-idl.c:742) ==20019== by 0x508128: ovsdb_idl_process_msg (ovsdb-idl.c:831) ==20019== by 0x508128: ovsdb_idl_run (ovsdb-idl.c:915) ==20019== by 0x4106D9: bridge_run (bridge.c:2977) ==20019== by 0x40719C: main (ovs-vswitchd.c:127) ==20019== ==20019== LEAK SUMMARY: ==20019== definitely lost: 296 bytes in 1 blocks ==20019== indirectly lost: 13,587 bytes in 10 blocks ==20019== possibly lost: 0 bytes in 0 blocks ==20019== still reachable: 43,563 bytes in 440 blocks ==20019== suppressed: 288 bytes in 1 blocks ....
The problem is that table records maintained by database which is going to be destroyed with ovsdb_idl_db_destroy() function are not destroyed. Signed-off-by: Damijan Skvarc <[email protected]> --- lib/ovsdb-idl.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/lib/ovsdb-idl.c b/lib/ovsdb-idl.c index 7545e43..d64d8c0 100644 --- a/lib/ovsdb-idl.c +++ b/lib/ovsdb-idl.c @@ -239,6 +239,7 @@ static bool ovsdb_idl_check_server_db(struct ovsdb_idl *); static void ovsdb_idl_send_monitor_request(struct ovsdb_idl *, struct ovsdb_idl_db *, enum ovsdb_idl_monitor_method); +static void ovsdb_idl_db_clear(struct ovsdb_idl_db *db); struct ovsdb_idl { struct ovsdb_idl_db server; @@ -581,8 +582,9 @@ ovsdb_idl_destroy(struct ovsdb_idl *idl) if (idl) { ovsdb_idl_clear(idl); jsonrpc_session_close(idl->session); - + ovsdb_idl_db_clear(&idl->server); ovsdb_idl_db_destroy(&idl->server); + ovsdb_idl_db_clear(&idl->data); ovsdb_idl_db_destroy(&idl->data); json_destroy(idl->request_id); free(idl->remote); -- 2.7.4 _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
