[ovs-dev] [PATCH] ovsdb: Fix memory leak reported by valgrind.

William Tu u9012063 at gmail.com
Mon Jul 25 14:07:35 UTC 2016


Looks good to me, thanks for fixing it!

Acked-by: William Tu <u9012063 at gmail.com>

On Mon, Jul 25, 2016 at 1:00 AM, Liran Schour <lirans at il.ibm.com> wrote:
> Destroy shash on destroy of session's condition structure.
> Reported here: http://openvswitch.org/pipermail/dev/2016-July/075968.html
>
> Signed-off-by: Liran Schour <lirans at il.ibm.com>
> ---
>  ovsdb/monitor.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/ovsdb/monitor.c b/ovsdb/monitor.c
> index 99083e1..9a6fbf5 100644
> --- a/ovsdb/monitor.c
> +++ b/ovsdb/monitor.c
> @@ -617,6 +617,7 @@ ovsdb_monitor_session_condition_destroy(
>          shash_delete(&condition->tables, node);
>          free(mtc);
>      }
> +    shash_destroy(&condition->tables);
>      free(condition);
>  }
>
> --
> 2.1.4
>



More information about the dev mailing list