We were leaking the ovn_version string as reported by Address
Sanitizers:
Direct leak of 22 byte(s) in 1 object(s) allocated from:
malloc (/workspace/ovn-tmp/controller-vtep/ovn-controller-vtep+0x239ddf)
xmalloc__ lib/util.c:141
xmalloc lib/util.c:176
xvasprintf lib/util.c:233
xasprintf lib/util.c:374
ovn_get_internal_version lib/ovn-util.c:944
main controller-vtep/ovn-controller-vtep.c:178
Fixes: 5bd7e7a55d8e ("vtep: provide option to check ovn-controller-vtep and
ovn-northd versions")
Signed-off-by: Ales Musil <[email protected]>
---
controller-vtep/ovn-controller-vtep.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/controller-vtep/ovn-controller-vtep.c
b/controller-vtep/ovn-controller-vtep.c
index 698511482..c5d7f599f 100644
--- a/controller-vtep/ovn-controller-vtep.c
+++ b/controller-vtep/ovn-controller-vtep.c
@@ -257,6 +257,7 @@ main(int argc, char *argv[])
ovsdb_idl_loop_destroy(&vtep_idl_loop);
ovsdb_idl_loop_destroy(&ovnsb_idl_loop);
+ free(ovn_version);
free(ovnsb_remote);
free(vtep_remote);
free(default_db_);
--
2.52.0
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev