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

"It does what it says on the lid" - do_init_tun() will allocate a new
struct tuntap, so if we have one beforehand, it needs free()ing.  

I have not actively *tested* it, as I have no test rig for building 
Android binaries - OTOH, if the one who gets all the blame for Android 
says it's ok, I'm going to believe it (no other platform affected).

Your patch has been applied to the master and release/2.4 branch
(2.4 mostly due to "it's a bugfix, it's a single platform, and 
to keep the code similar for future changes").

commit da3f583f30a4b2be9cc5501874373fc4f627158d (master)
commit 21a307dd7252a066f204dee52a4cf569a4a48a1d (release/2.4)
Author: Arne Schwabe
Date:   Wed Sep 12 13:07:01 2018 +0200

     Properly free tuntap struct on android when emulating persist-tun

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