On Tue, Jan 6, 2026 at 2:41 PM Ales Musil <[email protected]> wrote:
> 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
>
>
Recheck-request: github-robot-_Build_and_Test
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
