From cba5d1489bf89ff81472986b18b9b13c5b5f8748 Mon Sep 17 00:00:00 2001 From: Alin Serdean Date: Mon, 6 Nov 2017 17:33:33 +0200 Subject: datapath-windows: Use only non executable memory Use only non-executable memory when using MmGetSystemAddressForMdlSafe. Introduce a new function called OvsGetMdlWithLowPriority for readability. Found using WDK 10 static code analysis. Signed-off-by: Alin Gabriel Serdean Acked-by: Shashank Ram --- datapath-windows/ovsext/BufferMgmt.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'datapath-windows/ovsext/BufferMgmt.c') diff --git a/datapath-windows/ovsext/BufferMgmt.c b/datapath-windows/ovsext/BufferMgmt.c index ff4f9bbb5..03470d787 100644 --- a/datapath-windows/ovsext/BufferMgmt.c +++ b/datapath-windows/ovsext/BufferMgmt.c @@ -1157,7 +1157,7 @@ FixFragmentHeader(PNET_BUFFER nb, UINT16 fragmentSize, mdl = NET_BUFFER_FIRST_MDL(nb); - bufferStart = (PUINT8)MmGetSystemAddressForMdlSafe(mdl, LowPagePriority); + bufferStart = (PUINT8)OvsGetMdlWithLowPriority(mdl); if (!bufferStart) { return NDIS_STATUS_RESOURCES; } @@ -1215,7 +1215,7 @@ FixSegmentHeader(PNET_BUFFER nb, UINT16 segmentSize, UINT32 seqNumber, mdl = NET_BUFFER_FIRST_MDL(nb); - bufferStart = (PUINT8)MmGetSystemAddressForMdlSafe(mdl, LowPagePriority); + bufferStart = (PUINT8)OvsGetMdlWithLowPriority(mdl); if (!bufferStart) { return NDIS_STATUS_RESOURCES; } @@ -1521,8 +1521,8 @@ OvsAllocateNBLFromBuffer(PVOID context, nb = NET_BUFFER_LIST_FIRST_NB(nbl); mdl = NET_BUFFER_CURRENT_MDL(nb); - data = (PUINT8)MmGetSystemAddressForMdlSafe(mdl, LowPagePriority) + - NET_BUFFER_CURRENT_MDL_OFFSET(nb); + data = (PUINT8)OvsGetMdlWithLowPriority(mdl) + + NET_BUFFER_CURRENT_MDL_OFFSET(nb); if (!data) { OvsCompleteNBL(switchContext, nbl, TRUE); return NULL; -- cgit v1.2.1