Ben Pfaff <b...@ovn.org> wrote on 09/01/2016 12:02:53 PM:

> From: Ben Pfaff <b...@ovn.org>
> To: dev@openvswitch.org
> Cc: Ben Pfaff <b...@ovn.org>, Ryan Moats/Omaha/IBM@IBMUS
> Date: 09/01/2016 12:04 PM
> Subject: [PATCH] ovn-controller: Fix memory leak in
> recv_S_TLV_TABLE_REQUESTED().
>
> Nothing freed 'reply'.  This fixes the problem.
>
> Most of this patch is moving coding around.  The essential change is that
> breaking the code that works with 'reply' out into a separate function
> makes it possible to catch all paths out of the function so that 'reply'
> can be freed in one place.
>
> Reported-by: Ryan Moats <rmo...@us.ibm.com>
> Signed-off-by: Ben Pfaff <b...@ovn.org>
> ---

That appears to do the trick as well and it is cleaner, so ... :)

Acked-by: Ryan Moats <rmo...@us.ibm.com>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to