diff options
author | Sean Bright <sean@malleable.com> | 2009-08-17 18:18:16 +0000 |
---|---|---|
committer | Sean Bright <sean@malleable.com> | 2009-08-17 18:18:16 +0000 |
commit | 0c99f79a7621ed1fa6aef0748735b5d70ceac7ed (patch) | |
tree | 59239f12c6228049c22dbf371b2e1ab173e6479a /main | |
parent | 36b38525a168716efb22a8f1da00d3dec88156d4 (diff) |
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: https://origsvn.digium.com/svn/asterisk/trunk@212574 65c4cc65-6c06-0410-ace0-fbb531ad65f3
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 02cf4f4fc..ea8c10d45 100644 --- a/main/logger.c +++ b/main/logger.c @@ -539,7 +539,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); } c = ast_channel_release(c); |