diff mbox

[ovs-dev] datapath-windows: Remove dead code from PacketIO

Message ID 20161208175709.11792-1-aserdean@cloudbasesolutions.com
State Accepted
Delegated to: Guru Shetty
Headers show

Commit Message

Alin Serdean Dec. 8, 2016, 5:57 p.m. UTC 
Assigning value to 'nativeNbls' has no effect outside the function and
the variable is not used inside the function.

Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com>
---
 datapath-windows/ovsext/PacketIO.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Ben Pfaff Dec. 21, 2016, 10:43 p.m. UTC | #1 
On Thu, Dec 08, 2016 at 05:57:19PM +0000, Alin Serdean wrote:
> Assigning value to 'nativeNbls' has no effect outside the function and
> the variable is not used inside the function.
> 
> Signed-off-by: Alin Gabriel Serdean <aserdean@cloudbasesolutions.com>

Applied, thanks!
diff mbox

Patch

diff --git a/datapath-windows/ovsext/PacketIO.c b/datapath-windows/ovsext/PacketIO.c
index a0ddc3d..e30a0c1 100644
--- a/datapath-windows/ovsext/PacketIO.c
+++ b/datapath-windows/ovsext/PacketIO.c
@@ -193,7 +193,6 @@  OvsAppendNativeForwardedPacket(POVS_SWITCH_CONTEXT switchContext,
     NDIS_STRING filterReason;
 
     *nativeNbls = curNbl;
-    nativeNbls = &(curNbl->Next);
 
     ctx = OvsInitExternalNBLContext(switchContext, curNbl, isRecv);
     if (ctx == NULL) {