I like that :-) - less code, less malloc()ing, less gcs around and 
more correct at that.  

I have also briefly reviewed the rest of the series, and everything looks 
good - but since Antonio has already ACKed everything, I leave that honour
to him.

Your patch has been applied to the master branch.

(This is refactoring, according to branch rules it only goes to 2.4 in
exceptionary circumstances, like "another patch depending on it" - but since
tls-crypt-v2 is also intended for master only, this is good enough for now)

commit d2342067d95621b130dd3985a077872b032d53d2
Author: Steffan Karger
Date:   Wed Nov 1 23:03:39 2017 +0100

     pf: clean up temporary files if plugin init fails

     Signed-off-by: Steffan Karger <steffan.kar...@fox-it.com>
     Acked-by: Antonio Quartulli <anto...@openvpn.net>
     Message-Id: <20171101220342.14648-2-stef...@karger.me>
     URL: 
https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg15705.html
     Signed-off-by: Gert Doering <g...@greenie.muc.de>


--
kind regards,

Gert Doering


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Reply via email to