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/Vxlan.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/Vxlan.c')
-rw-r--r-- | datapath-windows/ovsext/Vxlan.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/datapath-windows/ovsext/Vxlan.c b/datapath-windows/ovsext/Vxlan.c index 47c734e1e..09809d397 100644 --- a/datapath-windows/ovsext/Vxlan.c +++ b/datapath-windows/ovsext/Vxlan.c @@ -244,8 +244,7 @@ OvsDoEncapVxlan(POVS_VPORT_ENTRY vport, } curMdl = NET_BUFFER_CURRENT_MDL(curNb); - bufferStart = (PUINT8)MmGetSystemAddressForMdlSafe(curMdl, - LowPagePriority); + bufferStart = (PUINT8)OvsGetMdlWithLowPriority(curMdl); if (!bufferStart) { status = NDIS_STATUS_RESOURCES; goto ret_error; @@ -421,8 +420,8 @@ OvsDecapVxlan(POVS_SWITCH_CONTEXT switchContext, curNbl = *newNbl; curNb = NET_BUFFER_LIST_FIRST_NB(curNbl); curMdl = NET_BUFFER_CURRENT_MDL(curNb); - bufferStart = (PUINT8)MmGetSystemAddressForMdlSafe(curMdl, LowPagePriority) + - NET_BUFFER_CURRENT_MDL_OFFSET(curNb); + bufferStart = (PUINT8)OvsGetMdlWithLowPriority(curMdl) + + NET_BUFFER_CURRENT_MDL_OFFSET(curNb); if (!bufferStart) { status = NDIS_STATUS_RESOURCES; goto dropNbl; |