[asterisk-commits] mnicholson: branch 1.6.1 r168015 - in /branches/1.6.1: ./ main/logger.c

[SVN commits to the Asterisk project]

Author: mnicholson
Date: Fri Jan  9 10:41:28 2009
New Revision: 168015

URL: http://svn.digium.com/view/asterisk?view=rev&rev=168015
Log:
Merged revisions 168014 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

........
  r168014 | mnicholson | 2009-01-09 10:32:34 -0600 (Fri, 09 Jan 2009) | 5 lines
  
  Use ast_safe_system() in logger.c instead of system()
  
  (closes issue #14194)
  Reported by: pabelanger
........

Modified:
    branches/1.6.1/   (props changed)
    branches/1.6.1/main/logger.c

Propchange: branches/1.6.1/
------------------------------------------------------------------------------
Binary property 'trunk-merged' - no diff available.

Modified: branches/1.6.1/main/logger.c
URL: http://svn.digium.com/view/asterisk/branches/1.6.1/main/logger.c?view=diff&rev=168015&r1=168014&r2=168015
==============================================================================
--- branches/1.6.1/main/logger.c (original)
+++ branches/1.6.1/main/logger.c Fri Jan  9 10:41:28 2009
@@ -74,6 +74,7 @@
 #include "asterisk/threadstorage.h"
 #include "asterisk/strings.h"
 #include "asterisk/pbx.h"
+#include "asterisk/app.h"
 
 #if defined(__linux__) && !defined(__NR_gettid)
 #include <asm/unistd.h>
@@ -576,8 +577,8 @@
 		char buf[512];
 		pbx_builtin_setvar_helper(c, "filename", filename);
 		pbx_substitute_variables_helper(c, exec_after_rotate, buf, sizeof(buf));
-		if (system(buf) < 0) {
-			ast_log(LOG_WARNING, "system() failed for '%s': %s\n", buf, strerror(errno));
+		if (ast_safe_system(buf) != -1) {
+			ast_log(LOG_WARNING, "error executing '%s'\n", buf);
 		}
 		ast_channel_free(c);
 	}