diff options
| author | seanbright <seanbright@f38db490-d61c-443f-a65b-d21fe96a405b> | 2009-08-17 18:21:40 +0000 |
|---|---|---|
| committer | seanbright <seanbright@f38db490-d61c-443f-a65b-d21fe96a405b> | 2009-08-17 18:21:40 +0000 |
| commit | 87303b5b2db577c47785c772fe46f82100a13196 (patch) | |
| tree | 38eaf1835054a66706c01cf1e5a66e0bc1f8be79 /main | |
| parent | 3ad6834e6d24390a9f2e388102eac8c709faf7f7 (diff) | |
Merged revisions 212574 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk
........
r212574 | seanbright | 2009-08-17 14:18:16 -0400 (Mon, 17 Aug 2009) | 8 lines
Correct the return value check for ast_safe_system.
The logic here was reversed as ast_safe_system returns -1 on error and not on
success. Fix suggested by reporter.
(closes issue #15667)
Reported by: loic
........
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.2@212580 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'main')
| -rw-r--r-- | main/logger.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/main/logger.c b/main/logger.c index b02cca694..8c0968ce6 100644 --- a/main/logger.c +++ b/main/logger.c @@ -575,7 +575,7 @@ static int rotate_file(const char *filename) char buf[512]; pbx_builtin_setvar_helper(c, "filename", filename); pbx_substitute_variables_helper(c, exec_after_rotate, buf, sizeof(buf)); - if (ast_safe_system(buf) != -1) { + if (ast_safe_system(buf) == -1) { ast_log(LOG_WARNING, "error executing '%s'\n", buf); } ast_channel_free(c); |