From: Lev Stipakov <l...@openvpn.net>
As MS documentation says:
> If lpOverlapped is NULL, lpBytesReturned cannot be NULL
While on Windows 10 passing NULL works by accident,
on Windows 7 it crashes.
Reported-by: kitsune1
Signed-off-by: Lev Stipakov <l...@openvpn.net>
---
src/openvpn/ring_buffer.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/openvpn/ring_buffer.c b/src/openvpn/ring_buffer.c
index 482e3336..d384f497 100644
--- a/src/openvpn/ring_buffer.c
+++ b/src/openvpn/ring_buffer.c
@@ -35,6 +35,7 @@ register_ring_buffers(HANDLE device,
{
struct tun_register_rings rr;
BOOL res;
+ DWORD bytes_returned;
ZeroMemory(&rr, sizeof(rr));
@@ -46,7 +47,8 @@ register_ring_buffers(HANDLE device,
rr.receive.ring_size = sizeof(receive_ring->data);
rr.receive.tail_moved = receive_tail_moved;
- res = DeviceIoControl(device, TUN_IOCTL_REGISTER_RINGS, &rr, sizeof(rr),
NULL, 0, NULL, NULL);
+ res = DeviceIoControl(device, TUN_IOCTL_REGISTER_RINGS, &rr, sizeof(rr),
+ NULL, 0, &bytes_returned, NULL);
return res == TRUE;
}
--
2.17.1
_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel
