diff mbox

[ovs-dev] datapath-windows: Fix a bug in OvsCreateNewNBLsFromMultipleNBs()

Message ID BY2PR0501MB21195E93AF728F4FED8BC6A7A2BB0@BY2PR0501MB2119.namprd05.prod.outlook.com
State Superseded
Headers show

Commit Message

Shashank Ram July 24, 2017, 5:05 p.m. UTC
I will hold onto this patch a bit longer. Noticed that there are other bugs that get exposed as a result of this bug fix. Will send out a v2.

Thanks,
Shashank
diff mbox

Patch

diff --git a/datapath-windows/ovsext/PacketIO.c b/datapath-windows/ovsext/PacketIO.c
index a90b556..d82d25d 100644
--- a/datapath-windows/ovsext/PacketIO.c
+++ b/datapath-windows/ovsext/PacketIO.c
@@ -525,11 +525,11 @@  OvsCreateNewNBLsFromMultipleNBs(POVS_SWITCH_CONTEXT switchContext,
         }
         lastNbl->Next = *nextNbl;
         *nextNbl = newNbls->Next;
+        (*curNbl)->Next = NULL;

         OvsCompleteNBL(switchContext, *curNbl, TRUE);

         *curNbl = newNbls;
-        (*curNbl)->Next = NULL;

         error = FALSE;
     } while (error);