From 864cae562c2c92dcc9481544463409f25572d011 Mon Sep 17 00:00:00 2001 From: seanbright Date: Mon, 17 Aug 2009 18:18:16 +0000 Subject: 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: http://svn.digium.com/svn/asterisk/trunk@212574 f38db490-d61c-443f-a65b-d21fe96a405b --- main/logger.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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); -- cgit v1.2.3