diff options
| author | Linus Nordberg <linus@nordu.net> | 2012-04-10 15:55:57 +0200 |
|---|---|---|
| committer | Linus Nordberg <linus@nordu.net> | 2012-04-12 17:26:13 +0200 |
| commit | 8b7224cfaf6961c48c018e6cfc500d577c8caaac (patch) | |
| tree | 9b0b5b56cc4e31b084976472a874d2504167d0c9 /radsecproxy.c | |
| parent | 39e90a8cfdfe5bc95d2d01aa680e328c3476ff33 (diff) | |
Don't treat exit 10 from dynamic scripts differently from any other non-zero code.
clientwr() should treat the dynamic lookup as a failure and will not
be any happier to know that the exact error was that it didn't resolv.
The script can do whatever logging is wanted.
That said, this commit also makes the scripts exit with 10 in order to
signal failure.
Diffstat (limited to 'radsecproxy.c')
| -rw-r--r-- | radsecproxy.c | 11 |
1 files changed, 3 insertions, 8 deletions
diff --git a/radsecproxy.c b/radsecproxy.c index 5248c41..d8500c1 100644 --- a/radsecproxy.c +++ b/radsecproxy.c @@ -2297,14 +2297,9 @@ int dynamicconfig(struct server *server) { } if (status) { - if (WEXITSTATUS(status) == 10) { - debug(DBG_INFO, "dynamicconfig: command signals empty config"); - } - else { - debug(DBG_INFO, "dynamicconfig: command exited with status %d", - WEXITSTATUS(status)); - goto errexit; - } + debug(DBG_INFO, "dynamicconfig: command exited with status %d", + WEXITSTATUS(status)); + goto errexit; } if (ok) |