From 86f663d7f4a63a4e82c7030fa96a9574c1e945a1 Mon Sep 17 00:00:00 2001 From: Jonathan Rose Date: Fri, 29 Mar 2013 16:23:03 +0000 Subject: [PATCH] app_voicemail: Add blank argument to externnotify if no context argument At least one call to run_externnotify provides a NULL context parameter and because the snprintf statement doesn't account for a NULL context parameter, it simply writes '(null)' to the arguments string instead. This patch makes it write two quotes back to back for that argument instead in the event of a NULL context. (closes issue ASTERISK-18207) Reported by: Barry L. Kline Patches: modified from patch-20130306 uploaded by Karsten Wemheuer (License 5930) git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@384325 65c4cc65-6c06-0410-ace0-fbb531ad65f3 --- apps/app_voicemail.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/apps/app_voicemail.c b/apps/app_voicemail.c index 3e34307e83..5f1663b982 100644 --- a/apps/app_voicemail.c +++ b/apps/app_voicemail.c @@ -5566,7 +5566,10 @@ static void run_externnotify(char *context, char *extension, const char *flag) if (inboxcount2(ext_context, &urgentvoicemails, &newvoicemails, &oldvoicemails)) { ast_log(AST_LOG_ERROR, "Problem in calculating number of voicemail messages available for extension %s\n", extension); } else { - snprintf(arguments, sizeof(arguments), "%s %s %s %d %d %d &", externnotify, context, extension, newvoicemails, oldvoicemails, urgentvoicemails); + snprintf(arguments, sizeof(arguments), "%s %s %s %d %d %d &", + externnotify, S_OR(context, "\"\""), + extension, newvoicemails, + oldvoicemails, urgentvoicemails); ast_debug(1, "Executing %s\n", arguments); ast_safe_system(arguments); } -- 2.47.3