ACK, making engine state explicit is good style - and it protects against crashes if calling win_wfp_uninit() twice without opening it in between again.
Your patch has been applied to the master branch. commit 895d75cf99bf8f3b95b01d6e19df30208cec27a9 Author: Selva Nair List-Post: openvpn-devel@lists.sourceforge.net Date: Sat Jun 4 16:09:05 2016 -0400 Set WFP engine handle to NULL in win_wfp_uninit() Signed-off-by: Selva Nair <selva.n...@gmail.com> Acked-by: Gert Doering <g...@greenie.muc.de> Message-Id: <1465070945-5426-1-git-send-email-selva.n...@gmail.com> URL: http://article.gmane.org/gmane.network.openvpn.devel/11789 Signed-off-by: Gert Doering <g...@greenie.muc.de> -- kind regards, Gert Doering