Your patch has been applied to the master branch.

I have "forward-logged" Steffan's ACK on the v6 patch as the v8 is
close enough, except for the "struct win_tun" change (which look 
reasonable),  plus Simon's ACK.

What I do not like is code that uses malloc() and then calls ZeroMemory()
right after.  calloc() exists.  Standard-based, less lines of code.

No reason to NAK this, but maybe this can be fixed in a followup patch
(and put on our styleguide somewhere?)...

Compile-tested on Ubuntu 16.04 / mingw (which worked fine).  No own
code review or any sort of testing done.

commit d5fc4bd41616ee01816df873cb6b6567872a29e2
Author: Lev Stipakov
Date:   Tue Dec 17 14:44:10 2019 +0200

     wintun: ring buffers based I/O

     Signed-off-by: Lev Stipakov <l...@openvpn.net>
     Acked-by: Steffan Karger <steffan.kar...@fox-it.com>
     Acked-by: Simon Rozman <si...@rozman.si>
     Message-Id: <20191217124410.81-1-lstipa...@gmail.com>
     URL: 
https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg19243.html
     Signed-off-by: Gert Doering <g...@greenie.muc.de>


--
kind regards,

Gert Doering



_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Reply via email to