[ovs-dev] [PATCH] datapath-windows: don't leak NBLs with multiple NBs
Alin Serdean
aserdean at cloudbasesolutions.com
Fri Oct 31 02:42:30 UTC 2014
Tested-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Acked-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
-----Mesaj original-----
De la: dev [mailto:dev-bounces at openvswitch.org] În numele Nithin Raju
Trimis: Thursday, October 30, 2014 10:18 PM
Către: dev at openvswitch.org
Subiect: [ovs-dev] [PATCH] datapath-windows: don't leak NBLs with multiple NBs
Currently, if we receive an NBL with multiple NBs from NDIS, we just
ASSERT() and not do anything. The right thing to do obviously is to process the NBL. This is a work in progress.
In the meantime, we should complete the NBL and not just leak it.
Signed-off-by: Nithin Raju <nithin at vmware.com>
---
datapath-windows/ovsext/PacketIO.c | 7 ++++++-
1 files changed, 6 insertions(+), 1 deletions(-)
diff --git a/datapath-windows/ovsext/PacketIO.c b/datapath-windows/ovsext/PacketIO.c
index 5223125..1af391b 100644
--- a/datapath-windows/ovsext/PacketIO.c
+++ b/datapath-windows/ovsext/PacketIO.c
@@ -218,7 +218,12 @@ OvsStartNBLIngress(POVS_SWITCH_CONTEXT switchContext,
curNb = NET_BUFFER_LIST_FIRST_NB(curNbl);
if (curNb->Next != NULL) {
/* XXX: This case is not handled yet. */
- ASSERT(FALSE);
+ RtlInitUnicodeString(&filterReason,
+ L"Dropping NBLs with multiple NBs");
+ OvsStartNBLIngressError(switchContext, curNbl,
+ sendCompleteFlags, &filterReason,
+ NDIS_STATUS_RESOURCES);
+ continue;
} else {
POVS_BUFFER_CONTEXT ctx;
OvsFlow *flow;
--
1.7.4.1
_______________________________________________
dev mailing list
dev at openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
More information about the dev
mailing list