[ovs-dev] [PATCH] datapath-windows: Solved memory leak in OVS datapath

Ben Pfaff blp at nicira.com
Thu Jul 2 15:36:23 UTC 2015


Thanks Alin and Sorin, applied to master and branch-2.4.

On Thu, Jul 02, 2015 at 07:24:04AM +0000, Alin Serdean wrote:
> Acked-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
> 
> 
> -----Mesaj original-----
> De la: dev [mailto:dev-bounces at openvswitch.org] În numele Sorin Vinturis
> Trimis: Thursday, July 2, 2015 9:53 AM
> Către: dev at openvswitch.org
> Subiect: [ovs-dev] [PATCH] datapath-windows: Solved memory leak in OVS datapath
> 
> When closing opened instances, make sure the user dump state is cleared.
> 
> Signed-off-by: Sorin Vinturis <svinturis at cloudbasesolutions.com>
> Reported-by: Sorin Vinturis <svinturis at cloudbasesolutions.com>
> Reported-at: https://github.com/openvswitch/ovs-issues/issues/90
> ---
> This patch should be applied both on master and branch 2.4.
> ---
>  datapath-windows/ovsext/Datapath.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/datapath-windows/ovsext/Datapath.c b/datapath-windows/ovsext/Datapath.c
> index d8024c8..4af909c 100644
> --- a/datapath-windows/ovsext/Datapath.c
> +++ b/datapath-windows/ovsext/Datapath.c
> @@ -588,6 +588,7 @@ OvsRemoveOpenInstance(PFILE_OBJECT fileObject)
>      OvsReleaseCtrlLock();
>      ASSERT(instance->eventQueue == NULL);
>      ASSERT (instance->packetQueue == NULL);
> +    FreeUserDumpState(instance);
>      OvsFreeMemoryWithTag(instance, OVS_DATAPATH_POOL_TAG);  }
>  
> --
> 1.9.0.msysgit.0
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev



More information about the dev mailing list