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).
This patch is to be applied both on master and on 2.4 branch. Signed-off-by: Sorin Vinturis <svintu...@cloudbasesolutions.com> Reported-by: Sorin Vinturis <svintu...@cloudbasesolutions.com> Reported-at: https://github.com/openvswitch/ovs-issues/issues/87 --- datapath-windows/ovsext/BufferMgmt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/datapath-windows/ovsext/BufferMgmt.c b/datapath-windows/ovsext/BufferMgmt.c index 5adbb25..83d6cde 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; } -- 1.9.0.msysgit.0 _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
