[ovs-git] [openvswitch/ovs] 4ef7f5: ovsdb-idl: Fix memory leak
GitHub
noreply at github.com
Thu Nov 2 21:21:51 UTC 2017
Branch: refs/heads/branch-2.8
Home: https://github.com/openvswitch/ovs
Commit: 4ef7f55e1bc6a2ac82a4ab582d2b8101524ca2f1
https://github.com/openvswitch/ovs/commit/4ef7f55e1bc6a2ac82a4ab582d2b8101524ca2f1
Author: Yifeng Sun <pkusunyifeng at gmail.com>
Date: 2017-11-02 (Thu, 02 Nov 2017)
Changed paths:
M lib/ovsdb-idl.c
Log Message:
-----------
ovsdb-idl: Fix memory leak
Valgrind testcase 2339 (ovn -- ipam connectivity) reports the leak below:
45 (32 direct, 13 indirect) bytes in 1 blocks are definitely lost in loss record 65 of 83
at 0x4C2DB8F: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
by 0x4A6D64: xmalloc (util.c:120)
by 0x49C847: shash_add_nocopy__ (shash.c:109)
by 0x49C847: shash_add_nocopy (shash.c:121)
by 0x49CA85: shash_add (shash.c:129)
by 0x49CA85: shash_add_once (shash.c:136)
by 0x4914B5: ovsdb_idl_create_index (ovsdb-idl.c:2067)
by 0x406C98: create_ovnsb_indexes (ovn-controller.c:568)
by 0x406C98: main (ovn-controller.c:619)
The leak happens when vsdb_idl_table is freed but its indexes are not freed.
Signed-off-by: Yifeng Sun <pkusunyifeng at gmail.com>
Signed-off-by: Ben Pfaff <blp at ovn.org>
More information about the git
mailing list