I have not specifically tested the scenarios that this is fixing (due to time constraints and really poor Internet $here). I totally trust Steffan's review and Richard's testing, though, to fix the problem observed.
I have run it through t_client tests (against git master servers about ~1 year old) and through the t_server tests (against 2.2/2.3/ 2.3-small/2.4 and roughly 4-weeks-old master clients) and these all pass. So it's definitely not breaking anything I'm currently testing :-) I have worked in Richard's most recent suggestions for the commit message and reworded it a bit more. If it does not make *any* sense anymore, it's all my fault. Your patch has been applied to the master and release/2.5 branch. commit 2ab0a92442dce1d82fcb9e2b305313ef668d40bf (master) commit a61883406f7caf39aab274979bb2d43d85804df8 (release/2.5) Author: Arne Schwabe Date: Fri Aug 14 10:06:19 2020 +0200 Fix client's poor man NCP fallback Signed-off-by: Arne Schwabe <a...@rfc2549.org> Acked-by: Steffan Karger <stef...@karger.me> Message-Id: <20200814080619.2108-1-a...@rfc2549.org> URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg20734.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