diff options
| author | Fabian Knittel <fabian.knittel@avona.com> | 2010-05-04 16:21:47 +0200 |
|---|---|---|
| committer | David Sommerseth <dazo@users.sourceforge.net> | 2010-05-04 23:12:15 +0200 |
| commit | c5b7923a2b0a94d702e1dad59438f7ee75971d3b (patch) | |
| tree | c08cf576c35f56433b3ca8b9c56ece194c12ede1 /contrib/openvpn-fwmarkroute-1.00 | |
| parent | Revamped the script-security warning logging (version 2) (diff) | |
| download | openvpn-c5b7923a2b0a94d702e1dad59438f7ee75971d3b.tar.xz | |
ssl.c: fix use of openvpn_run_script()'s return value
This patch fixes two bugs introduced in
commit 339f2a4d4b487afa53fa99d72c35b16f31e417d3
Author: David Sommerseth <dazo@users.sourceforge.net>
Date: Thu Apr 29 23:35:45 2010 +0200
David's patch replaced openvpn_execve() with openvpn_run_script() in two places,
but didn't adjust the return value handling. openvpn_run_script() returns true
or false, while openvpn_execve() returns the program's exit code.
Without the fix, the --tls-verify script and the --auth-user-pass-verify
script fail to run. (I noticed the latter, but haven't actually tested the
former.)
The return value handling is fine for the other places where
openvpn_run_script() is used, because those places previously used
openvpn_execve_check() (notice the "_check" suffix).
Signed-off-by: Fabian Knittel <fabian.knittel@avona.com>
Signed-off-by: David Sommerseth <dazo@users.sourceforge.net>
Acked-by: David Sommerseth <dazo@users.sourceforge.net>
Diffstat (limited to 'contrib/openvpn-fwmarkroute-1.00')
0 files changed, 0 insertions, 0 deletions