diff options
| author | Alin Serdean <aserdean@cloudbasesolutions.com> | 2017-11-06 17:33:33 +0200 |
|---|---|---|
| committer | Alin Gabriel Serdean <aserdean@ovn.org> | 2017-11-29 16:51:08 +0200 |
| commit | cba5d1489bf89ff81472986b18b9b13c5b5f8748 (patch) | |
| tree | 477eb1bed705fa20b7017692803515b25499f5ea /datapath-windows/ovsext/BufferMgmt.c | |
| parent | c8025aee4fa6a862d142dd8da780d45aaf0a868c (diff) | |
| download | openvswitch-cba5d1489bf89ff81472986b18b9b13c5b5f8748.tar.gz | |
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 <aserdean@cloudbasesolutions.com>
Acked-by: Shashank Ram <shashank08@gmail.com>
Diffstat (limited to 'datapath-windows/ovsext/BufferMgmt.c')
| -rw-r--r-- | datapath-windows/ovsext/BufferMgmt.c | 8 |
1 files changed, 4 insertions, 4 deletions
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; |