Acked-by: Gert Doering <g...@greenie.muc.de>

I follow the reasoning - since we set the MTU now, we do not need to
adjust the frame afterwards.  Logging the MTU might be useful (Windows
is weird, maybe it does not always "stick"...), so let's keep this in 
for the time being.

Spurious comment change in ssl.c removed from the patch.

I like the comment

 /* Some TUN/TAP drivers like to be ioctled for mtu after open */

"like to be ioctled", hah :-) - but why this is there is lost lore from
the long gone past...

I have test compiled this on mingw.


Your patch has been applied to the master branch.

commit c27868bfc2ad6ee21c28f4addb1c2fc003bfa61a
Author: Arne Schwabe
Date:   Tue Dec 7 18:01:56 2021 +0100

     Remove post_open_mtu code

     Signed-off-by: Arne Schwabe <a...@rfc2549.org>
     Acked-by: Gert Doering <g...@greenie.muc.de>
     Message-Id: <20211207170211.3275837-7-a...@rfc2549.org>
     URL: 
https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg23327.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