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

Reply via email to