On Aug 1, 2014, at 5:11 PM, Eitan Eliahu <elia...@vmware.com>
 wrote:

> BSOD while setting AllowManagementOS on $false #13
> Github Issue: #13
> 
> Signed-off-by: Eitan Eliahu <elia...@vmware.com>
> Reported-by: Alin Serdean <aserd...@cloudbasesolutions.com>
> ---
> datapath-windows/ovsext/OvsActions.c | 7 ++++++-
> 1 file changed, 6 insertions(+), 1 deletion(-)
> 
> diff --git a/datapath-windows/ovsext/OvsActions.c 
> b/datapath-windows/ovsext/OvsActions.c
> index 79fb50f..4a2c117 100644
> --- a/datapath-windows/ovsext/OvsActions.c
> +++ b/datapath-windows/ovsext/OvsActions.c
> @@ -581,7 +581,12 @@ OvsTunnelPortTx(OvsForwardingContext *ovsFwdCtx)
>      * Setup the source port to be the internal port to as to facilitate the
>      * second OvsLookupFlow.
>      */
> -    ASSERT(ovsFwdCtx->switchContext->internalVport);
> +    if (ovsFwdCtx->switchContext->internalVport == NULL) {
> +        OvsClearTunTxCtx(ovsFwdCtx);
> +        OvsCompleteNBLForwardingCtx(ovsFwdCtx,
> +            L"OVS-Dropped since internal port is absent");
> +        return NDIS_STATUS_FAILURE;
> +    }
>     ovsFwdCtx->srcVportNo =
>         ((POVS_VPORT_ENTRY)ovsFwdCtx->switchContext->internalVport)->portNo;

Acked-by: Nithin Raju <nit...@vmware.com>

thanks,
Nithin
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to