I'm not claiming to be an expert on Windows code and NRPT especially,
but I've stared hard at the code and this all seems to make sense
(including and especially memory and string handling).  Lev tested
this in detail, I just compile tested.

(The code change to openvpn itself is fairly straightforward and not
that windows specific.  The code change to the iservice is a tough one :-) ).

As discussed on IRC, sorry for the "%s" messup - but I would really
appreciate having "code fixes" and "new code" in separate commits, as
that makes reviewing & shipping the "fixes" quicker, and reviewing the
"new code" easier (less lines...).

Your patch has been applied to the master branch.

commit 0f3b7d17d565a422bfddc0f393b7712031e5bdbb
Author: Heiko Hund
Date:   Mon Apr 14 20:06:26 2025 +0200

     win: implement --dns option support with NRPT

     Signed-off-by: Heiko Hund <he...@ist.eigentlich.net>
     Acked-by: Lev Stipakov <lstipa...@gmail.com>
     Message-Id: <20250414180636.31936-1-g...@greenie.muc.de>
     URL: 
https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg31426.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