diff options
author | Sorin Vinturis <svinturis@cloudbasesolutions.com> | 2015-06-24 10:56:55 +0000 |
---|---|---|
committer | Ben Pfaff <blp@nicira.com> | 2015-06-24 10:59:57 -0700 |
commit | 885648d972f3d85c55cfb89e32773f1460bfe7ea (patch) | |
tree | 3cc2e13931a2ebe241ac72ad7f6b67b7991ed48c | |
parent | a0cba5c2f1040a7708a2a795461e5b9285161f65 (diff) | |
download | openvswitch-885648d972f3d85c55cfb89e32773f1460bfe7ea.tar.gz |
datapath-windows: Wrong cleanup of newly created multiple NBLs
Bug found in OvsPartialCopyToMultipleNBLs function in the cleanup part of
the code. Before completing the current NBL (newNbl) the NEXT link for the
following NBL (firstNbl) was broken, instead of the current one (newNbl).
Signed-off-by: Sorin Vinturis <svinturis@cloudbasesolutions.com>
Reported-by: Sorin Vinturis <svinturis@cloudbasesolutions.com>
Reported-at: https://github.com/openvswitch/ovs-issues/issues/87
Acked-by: Nithin Raju <nithin@vmware.com>
Signed-off-by: Ben Pfaff <blp@nicira.com>
-rw-r--r-- | datapath-windows/ovsext/BufferMgmt.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/datapath-windows/ovsext/BufferMgmt.c b/datapath-windows/ovsext/BufferMgmt.c index 5adbb2518..83d6cdee3 100644 --- a/datapath-windows/ovsext/BufferMgmt.c +++ b/datapath-windows/ovsext/BufferMgmt.c @@ -881,7 +881,7 @@ cleanup: newNbl = firstNbl; while (newNbl) { firstNbl = NET_BUFFER_LIST_NEXT_NBL(newNbl); - NET_BUFFER_LIST_NEXT_NBL(firstNbl) = NULL; + NET_BUFFER_LIST_NEXT_NBL(newNbl) = NULL; OvsCompleteNBL(context, newNbl, TRUE); newNbl = firstNbl; } |