Hello,

On Wed, Mar 6, 2019 at 4:23 AM Han Zhou <[email protected]> wrote:

> From: Han Zhou <[email protected]>
>
> Reported by Address Sanitizer.
>
> Signed-off-by: Han Zhou <[email protected]>
>

I think this fixes:

Fixes: 5e07b8f93f03 ("ovsdb-idl: New function
ovsdb_idl_create_unconnected().")

---
>  lib/ovsdb-idl.c | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/lib/ovsdb-idl.c b/lib/ovsdb-idl.c
> index 8cfb201..554bf76 100644
> --- a/lib/ovsdb-idl.c
> +++ b/lib/ovsdb-idl.c
> @@ -571,6 +571,9 @@ ovsdb_idl_destroy(struct ovsdb_idl *idl)
>          ovsdb_idl_db_destroy(&idl->server);
>          ovsdb_idl_db_destroy(&idl->data);
>          json_destroy(idl->request_id);
> +        if (idl->remote) {
> +            free(idl->remote);
> +        }
>

Testing for null is unnecessary.

         free(idl);
>      }
>  }
>


-- 
David Marchand
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to