[Asterisk-code-review] app voicemail: Clear voice mailbox in MailboxExists and MAIL... (asterisk[11])

Joshua Colp asteriskteam at digium.com
Wed Oct 26 08:16:25 CDT 2016


Joshua Colp has uploaded a new change for review. ( https://gerrit.asterisk.org/4186 )

Change subject: app_voicemail: Clear voice mailbox in MailboxExists and MAILBOX_EXISTS.
......................................................................

app_voicemail: Clear voice mailbox in MailboxExists and MAILBOX_EXISTS.

When executing the MailboxExists dialplan application and
MAILBOX_EXISTS dialplan function the passed in temporary voice
mailbox was not cleared, causing it to try to free garbage.

ASTERISK-26503 #close

Change-Id: Ie21ccfa1b80b9c59318e596f6b8e17da2b5a7cb3
---
M apps/app_voicemail.c
1 file changed, 2 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/86/4186/1

diff --git a/apps/app_voicemail.c b/apps/app_voicemail.c
index 4f8302b..cc931f1 100644
--- a/apps/app_voicemail.c
+++ b/apps/app_voicemail.c
@@ -12050,6 +12050,7 @@
 		context++;
 	}
 
+	memset(&svm, 0, sizeof(svm));
 	vmu = find_user(&svm, context, args.mbox);
 	if (vmu) {
 		pbx_builtin_setvar_helper(chan, "VMBOXEXISTSSTATUS", "SUCCESS");
@@ -12081,6 +12082,7 @@
 		ast_log(AST_LOG_WARNING, "MAILBOX_EXISTS is deprecated.  Please use ${VM_INFO(%s,exists)} instead.\n", args);
 	}
 
+	memset(&svm, 0, sizeof(svm));
 	vmu = find_user(&svm, ast_strlen_zero(arg.context) ? "default" : arg.context, arg.mbox);
 	ast_copy_string(buf, vmu ? "1" : "0", len);
 	free_user(vmu);

-- 
To view, visit https://gerrit.asterisk.org/4186
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie21ccfa1b80b9c59318e596f6b8e17da2b5a7cb3
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 11
Gerrit-Owner: Joshua Colp <jcolp at digium.com>



More information about the asterisk-code-review mailing list