summaryrefslogtreecommitdiff
path: root/main/logger.c
diff options
context:
space:
mode:
authorMatthew Nicholson <mnicholson@digium.com>2009-01-09 16:32:34 +0000
committerMatthew Nicholson <mnicholson@digium.com>2009-01-09 16:32:34 +0000
commitf42c44449ae48730acbacb82efd76831cd44022d (patch)
tree6158243dbfac0cf6dd8ddac7d43fdd378ca34ae7 /main/logger.c
parent1253c8aa0dd616aa5dffb82e65d311d010462d1e (diff)
Use ast_safe_system() in logger.c instead of system()
(closes issue #14194) Reported by: pabelanger git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@168014 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'main/logger.c')
-rw-r--r--main/logger.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/main/logger.c b/main/logger.c
index 8a577ed38..1be97fc87 100644
--- a/main/logger.c
+++ b/main/logger.c
@@ -74,6 +74,7 @@ static int syslog_level_map[] = {
#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 @@ 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 (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);
}