kenmirrors/asterisk
1
0
Fork 0
mirror of https://github.com/asterisk/asterisk.git synced 2025-11-08 10:58:15 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merged revisions 212574 via svnmerge from

Browse Source 
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: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@212578 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
Sean Bright
2009-08-17 18:21:36 +00:00
parent 19f97c9782
commit e15112b47c
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
main/logger.c
View File

@@ -522,7 +522,7 @@ static int rotate_file(const char *filename)
char buf[512]; char buf[512];
pbx_builtin_setvar_helper(c, "filename", filename); pbx_builtin_setvar_helper(c, "filename", filename);
pbx_substitute_variables_helper(c, exec_after_rotate, buf, sizeof(buf)); 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_log(LOG_WARNING, "error executing '%s'\n", buf);
} }
ast_channel_free(c); ast_channel_free(c);