[Openvpn-devel] [PATCH applied] Re: Fix documentation of tls-verify script argument

2019-02-28 Thread Gert Doering
Your patch has been applied to the master and release/2.4 branch. Thanks. commit a35d517f4893b3967783ad7926a7edd08f9f59d3 (master) commit 4e36eb913dfe5b4e4d9c0c0637f480c5b9f63e33 (release/2.4) Author: Thomas Quinot Date: Fri Feb 1 21:43:48 2019 +0100 Fix documentation of tls-verify script

[Openvpn-devel] [PATCH applied] Re: Disambiguate thread local storage references from TLS

2019-02-28 Thread Gert Doering
20190117155829.ga92...@greenie.muc.de> References: <20190224181252.43996-1-si...@rozman.si 20190117155829.ga92...@greenie.muc.de> Acked-by: Gert Doering "Does what I asked for and looks reasonable" ;-) - thanks. Your patch has been applied to the master branch. commit f75351da2974dac10baca5fa8

[Openvpn-devel] [PATCH applied] Re: Add user manual and developer notes URL for tapctl.exe

2019-02-28 Thread Gert Doering
20190117155829.ga92...@greenie.muc.de> References: <20190224181504.38328-1-si...@rozman.si 20190117155829.ga92...@greenie.muc.de> Acked-by: Gert Doering Trival but useful :) Your patch has been applied to the master branch. commit 890f9a85ca86fbba678fddb2e589a0180a6f1784 Author: Simon Rozman D

[Openvpn-devel] [PATCH applied] Re: Add NULL checks

2019-02-28 Thread Gert Doering
20190117155829.ga92...@greenie.muc.de> References: <20190224181400.42524-1-si...@rozman.si 20190117155829.ga92...@greenie.muc.de> Acked-by: Gert Doering Thanks :-) (not only NULL checks, but calloc() as well!) Test built on ubuntu1604/mingw. Your patch has been applied to the master branch.

Re: [Openvpn-devel] [PATCH applied] Re: Disambiguate thread local storage references from TLS

2019-02-28 Thread Gert Doering
Hi, "whatever happened here", this was supposed to thread... gert On Thu, Feb 28, 2019 at 03:54:15PM +0100, Gert Doering wrote: > 20190117155829.ga92...@greenie.muc.de> > References: <20190224181252.43996-1-si...@rozman.si > 20190117155829.ga92...@greenie.muc.de> > > Acked-by: Gert Doering >

[Openvpn-devel] [PATCH applied] Re: Add NULL checks

2019-02-28 Thread Gert Doering
(Re-send with proper references - my mailer and my git tools hate each other today) Acked-by: Gert Doering Thanks :-) (not only NULL checks, but calloc() as well!) Test built on ubuntu1604/mingw. Your patch has been applied to the master branch. commit fbb5d6c56b5586ffd3731fc76e706131b635be

[Openvpn-devel] [PATCH applied] Re: Add user manual and developer notes URL for tapctl.exe

2019-02-28 Thread Gert Doering
(Re-send with proper references - my mailer and my git tools hate each other today) Acked-by: Gert Doering Trival but useful :) Your patch has been applied to the master branch. commit 890f9a85ca86fbba678fddb2e589a0180a6f1784 Author: Simon Rozman Date: Sun Feb 24 19:15:04 2019 +0100 A

[Openvpn-devel] [PATCH applied] Re: Disambiguate thread local storage references from TLS

2019-02-28 Thread Gert Doering
(Re-send with proper references - my mailer and my git tools hate each other today) Acked-by: Gert Doering "Does what I asked for and looks reasonable" ;-) - thanks. Your patch has been applied to the master branch. commit f75351da2974dac10baca5fa83c18be7f27c73cf Author: Simon Rozman Date:

[Openvpn-devel] [PATCH applied] Re: Refactor OpenVPNService state detection code

2019-02-28 Thread Gert Doering
Acked-by: Gert Doering (Basically just reworking if/goto into if/else, plus more verbose comments. "Git show -w --color-moved=zebra" helps in seeing "nothing really changed") Your patch has been applied to the master branch. commit 197f8076e43534f3dfe50e3c638143c38f4bad7f Author: Simon Rozman D

Re: [Openvpn-devel] [PATCH applied] Re: Strip _stdcall suffixes (@nn) for 32-bit builds

2019-02-28 Thread Gert Doering
Hi, never replied to this one... On Sun, Jan 20, 2019 at 02:54:01PM +, Simon Rozman wrote: [..] > Wow, this made a nice text to start the developer documentation of MSI > setup. If it was understandable enough. :) It was understandable and matched what I had to guess out of reading quite a b

[Openvpn-devel] [PATCH applied] Re: Add developer notes URL for openvpnmsica.dll

2019-02-28 Thread Gert Doering
Acked-by: Gert Doering Trivial change, but very useful :-) - thanks. Your patch has been applied to the master branch. commit 5a05352f0ae06d2f79072fdc86370f279ed84074 Author: Simon Rozman Date: Sun Feb 24 19:55:00 2019 +0100 Add developer notes URL for openvpnmsica.dll Signed-off-

[Openvpn-devel] [PATCH applied] Re: Limit tapctl.exe and openvpnmsica.dll to TAP-Windows6 adapters only

2019-02-28 Thread Gert Doering
Acked-by: Gert Doering Thanks for the change. I have *not* tested it, just test compiled it (the usual ubuntu 16.04/mingw combo), but staring at the code, this look reasonable. Actually the change seems to make the code *more simple* (in FindTAPInterfaces() ) which is always welcome... Your pa

[Openvpn-devel] [PATCH applied] Re: cleanup: Remove RPM openvpn.spec build approach

2019-02-28 Thread Gert Doering
Acked-by: Gert Doering .. for the reasons given (as this is mostly removing non-code files, easy to review). Thanks for the documentation pointers. I have tested with "make distcheck" that autoconf still agrees with you on "which files should be there for packaging" (it does). Your patch has

[Openvpn-devel] [PATCH applied] Re: Better error message when script fails due to script-security setting

2019-02-28 Thread Gert Doering
Acked-by: Gert Doering Thanks. I have not tested the various error conditions, but the code makes sense to me and it passes compile and t_client tests (which use --up here), so it's not breaking stuff in fundamental and non-obvious ways. (Since David was mostly happy with the v1 approach except

Re: [Openvpn-devel] [PATCH applied] Re: Better error message when script fails due to script-security setting

2019-02-28 Thread Selva Nair
Hi, On Thu, Feb 28, 2019 at 12:25 PM Gert Doering wrote: > Acked-by: Gert Doering > > Thanks. I have not tested the various error conditions, but the code > makes sense to me and it passes compile and t_client tests (which use > --up here), so it's not breaking stuff in fundamental and non-obv

Re: [Openvpn-devel] [PATCH applied] Re: Better error message when script fails due to script-security setting

2019-02-28 Thread Gert Doering
Hi, On Thu, Feb 28, 2019 at 12:33:46PM -0500, Selva Nair wrote: > On Thu, Feb 28, 2019 at 12:25 PM Gert Doering wrote: > > > Your patch has been applied to the master branch - it won't apply to > > 2.4 as we have no "run_command.*" there yet. Shall we cherrypick > > bf97c00f7dba4 and 253f015558

[Openvpn-devel] [PATCH] Copy one byte less in strncpynt()

2019-02-28 Thread Gert Doering
While the existing code is not wrong and will never cause an overflow, it will copy (on a too-long source string) "maxlen" bytes to dest, and then overwrite the last byte just copied with "0" - which causes a warning in gcc 9 about filling the target buffer "up to the end, with no room for a traili

Re: [Openvpn-devel] [PATCH] Copy one byte less in strncpynt()

2019-02-28 Thread Antonio Quartulli
Hi, On 28/02/2019 19:51, Gert Doering wrote: > While the existing code is not wrong and will never cause an overflow, > it will copy (on a too-long source string) "maxlen" bytes to dest, and > then overwrite the last byte just copied with "0" - which causes a > warning in gcc 9 about filling the t

[Openvpn-devel] [PATCH for-2.4] Better error message when script fails due to script-security setting

2019-02-28 Thread selva . nair
From: Selva Nair - Add a new return value (-2) for openvpn_execve() when external program execution is not allowed due to a low script-security setting. - Add a corresponding error message Errors and warnings in such cases will now display as "WARNING: failed running command () :" followed