[svn-commits] seanbright: branch 1.6.1 r212579 - /branches/1.6.1/main/logger.c

[SVN commits to the Digium repositories]

Author: seanbright
Date: Mon Aug 17 13:21:38 2009
New Revision: 212579

URL: http://svn.asterisk.org/svn-view/asterisk?view=rev&rev=212579
Log:
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
........

Modified:
    branches/1.6.1/main/logger.c

Modified: branches/1.6.1/main/logger.c
URL: http://svn.asterisk.org/svn-view/asterisk/branches/1.6.1/main/logger.c?view=diff&rev=212579&r1=212578&r2=212579
==============================================================================
--- branches/1.6.1/main/logger.c (original)
+++ branches/1.6.1/main/logger.c Mon Aug 17 13:21:38 2009
@@ -571,7 +571,7 @@
 		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);