> On Sep 2, 2015, at 6:37 AM, Sorin Vinturis <svintu...@cloudbasesolutions.com> > wrote: > > Signed-off-by: Sorin Vinturis <svintu...@cloudbasesolutions.com> > --- > datapath-windows/ovsext/Vport.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/datapath-windows/ovsext/Vport.c b/datapath-windows/ovsext/Vport.c > index 6b74ec2..38cf878 100644 > --- a/datapath-windows/ovsext/Vport.c > +++ b/datapath-windows/ovsext/Vport.c > @@ -1267,7 +1267,9 @@ OvsRemoveTunnelVport(POVS_USER_PARAMS_CONTEXT > usrParamsCtx, > POVS_TUNFLT_INIT_CONTEXT tunnelContext = NULL; > PIRP irp = NULL; > > - tunnelContext = OvsAllocateMemory(sizeof(*tunnelContext)); > + tunnelContext = (POVS_TUNFLT_INIT_CONTEXT) > + OvsAllocateMemoryWithTag(sizeof(*tunnelContext), > + OVS_VPORT_POOL_TAG);
Do we need the typecast? The return is a VOID *, similar to what we had for OvsAllocateMemory(). Looks good otherwise. Acked-by: Nithin Raju <nit...@vmware.com> _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
