[ovs-dev] [PATCH 09/14] in-band: Fix memory leak in in_band_destroy().

Justin Pettit jpettit at nicira.com
Thu Apr 15 22:49:18 UTC 2010


Seems like it would be useful to actually free the data associated with a *_destroy function.

--Justin


On Apr 8, 2010, at 5:07 PM, Ben Pfaff wrote:

> ---
> ofproto/in-band.c |    1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/ofproto/in-band.c b/ofproto/in-band.c
> index 7e2fc53..1f9d422 100644
> --- a/ofproto/in-band.c
> +++ b/ofproto/in-band.c
> @@ -649,6 +649,7 @@ in_band_destroy(struct in_band *in_band)
>         netdev_close(in_band->local_netdev);
>         netdev_close(in_band->remote_netdev);
>         /* We don't own the rconn. */
> +        free(in_band);
>     }
> }
> 
> -- 
> 1.6.6.1
> 
> 
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev_openvswitch.org





More information about the dev mailing list