From c249e3e4817695e7b0f85644fa67b7f72667e988 Mon Sep 17 00:00:00 2001
From: Alin Serdean <aserdean@cloudbasesolutions.com>
Date: Mon, 6 Nov 2017 17:33:36 +0200
Subject: datapath-windows: Add assert to ethHdr in OvsActionMplsPush

`ethHdr` cannot be NULL because we did a partial copy before it.

Add an assert to keep the static analysis happy.

Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com>
Acked-by: Shashank Ram <shashank08@gmail.com>
---
 datapath-windows/ovsext/Actions.c | 1 +
 1 file changed, 1 insertion(+)

(limited to 'datapath-windows')

diff --git a/datapath-windows/ovsext/Actions.c b/datapath-windows/ovsext/Actions.c
index 0af4a660d..9bbc7879e 100644
--- a/datapath-windows/ovsext/Actions.c
+++ b/datapath-windows/ovsext/Actions.c
@@ -1237,6 +1237,7 @@ OvsActionMplsPush(OvsForwardingContext *ovsFwdCtx,
     ASSERT(mdlLen >= MPLS_HLEN);
 
     ethHdr = (EthHdr *)(bufferStart + curMdlOffset);
+    ASSERT(ethHdr);
     RtlMoveMemory(ethHdr, (UINT8*)ethHdr + MPLS_HLEN, sizeof(*ethHdr));
     ethHdr->Type = mpls->mpls_ethertype;
 
-- 
cgit v1.2.1