diff options
author | Nithin Raju <nithin@vmware.com> | 2016-05-19 15:31:48 -0700 |
---|---|---|
committer | Joe Stringer <joe@ovn.org> | 2016-05-25 10:25:17 -0700 |
commit | 57b99edd7eb351c76e3785effbf8463437202abf (patch) | |
tree | 9bcfc562915ecb0398825da5c121cfe705dc0785 /datapath-windows/ovsext | |
parent | bfac27150b4dedd6bb143299ef9bb1a31e448e62 (diff) | |
download | openvswitch-57b99edd7eb351c76e3785effbf8463437202abf.tar.gz |
datapath-windows: don't map output buffer in OVS_IOCTL_WRITE
The contract of OVS_IOCTL_WRITE is that write operations
will not need the output buffer. Only the input buffer
will be used in the IRP. So, better to not map the output
buffer at all.
Signed-off-by: Nithin Raju <nithin@vmware.com>
Acked-by: Paul-Daniel Boca <pboca@cloudbasesolutions.com>
Signed-off-by: Joe Stringer <joe@ovn.org>
Diffstat (limited to 'datapath-windows/ovsext')
-rw-r--r-- | datapath-windows/ovsext/Datapath.c | 13 |
1 files changed, 0 insertions, 13 deletions
diff --git a/datapath-windows/ovsext/Datapath.c b/datapath-windows/ovsext/Datapath.c index 1f89964de..e33027c90 100644 --- a/datapath-windows/ovsext/Datapath.c +++ b/datapath-windows/ovsext/Datapath.c @@ -869,19 +869,6 @@ OvsDeviceControl(PDEVICE_OBJECT deviceObject, goto done; } - /* - * Output buffer not mandatory but map it in case we have something - * to return to requester. - */ - if (outputBufferLen != 0) { - status = MapIrpOutputBuffer(irp, outputBufferLen, - sizeof *ovsMsg, &outputBuffer); - if (status != STATUS_SUCCESS) { - goto done; - } - ASSERT(outputBuffer); - } - ovsMsg = inputBuffer; ovsMsgLength = inputBufferLen; devOp = OVS_WRITE_DEV_OP; |