ACK, thanks.  Took us long enough.

Tested on Linux, does what it says (--verb 4 shows nice log messages,
with --allow-recursive-routing (and UDP), you get the expected CPU 
explosion instead).  

Passes all my t_client tests on client AND server, including --inetd
and p2p server.

I had to apply two of your changes manually - the option check in 
options.c and Changes.rst had changes in the preceding lines due to
other patches, so "git am" refused to apply it "just so".   Also, 
a small whitespace change was applied to proto.h

Your patch has been applied to the master and release/2.3 branch
(2.3 had conflicts in Changes.rst and options.[ch] - easily solved,
again by fixing contextual lines).

commit fa3e730e19a71419cb88a021b9a0333f088b39ac (master)
commit 7c930767c0b4ae38c4d886678f091a6077ed55a9 (release/2.3)
Author: Lev Stipakov
Date:   Sat Sep 17 12:47:32 2016 +0300

     Drop recursively routed packets

     Acked-by: Gert Doering <[email protected]>
     Message-Id: <[email protected]>
     URL: 
https://www.mail-archive.com/[email protected]/msg12476.html
     Signed-off-by: Gert Doering <[email protected]>


--
kind regards,

Gert Doering


------------------------------------------------------------------------------
Developer Access Program for Intel Xeon Phi Processors
Access to Intel Xeon Phi processor-based developer platforms.
With one year of Intel Parallel Studio XE.
Training and support from Colfax.
Order your platform today. http://sdm.link/xeonphi
_______________________________________________
Openvpn-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Reply via email to