From: Jeff Peeler Date: Wed, 21 Apr 2010 22:10:56 +0000 (+0000) Subject: Merged revisions 258433 via svnmerge from X-Git-Tag: 1.6.1.20-rc1~34 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=8448c7b538e154556642a2aafba04a8caa60fb4d;p=thirdparty%2Fasterisk.git Merged revisions 258433 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r258433 | jpeeler | 2010-04-21 16:56:09 -0500 (Wed, 21 Apr 2010) | 15 lines Merged revisions 258432 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r258432 | jpeeler | 2010-04-21 16:45:36 -0500 (Wed, 21 Apr 2010) | 8 lines Fix looping forever when no input received in certain voicemail menu scenarios. Specifically, prompting for an extension (when leaving or forwarding a message) or when prompting for a digit (when saving a message or changing folders). ABE-2122 SWP-1268 ........ ................ git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.1@258435 65c4cc65-6c06-0410-ace0-fbb531ad65f3 --- diff --git a/apps/app_voicemail.c b/apps/app_voicemail.c index 076229e64c..a5708b4029 100644 --- a/apps/app_voicemail.c +++ b/apps/app_voicemail.c @@ -6190,10 +6190,16 @@ static int get_folder(struct ast_channel *chan, int start) static int get_folder2(struct ast_channel *chan, char *fn, int start) { int res = 0; + int loops = 0; res = ast_play_and_wait(chan, fn); /* Folder name */ while (((res < '0') || (res > '9')) && - (res != '#') && (res >= 0)) { + (res != '#') && (res >= 0) && + loops < 4) { res = get_folder(chan, 0); + loops++; + } + if (loops == 4) { /* give up */ + return '#'; } return res; } @@ -6508,6 +6514,7 @@ static int forward_message(struct ast_channel *chan, char *context, struct vm_st int curmsg; char urgent_str[7] = ""; char tmptxtfile[PATH_MAX]; + int prompt_played = 0; #ifndef IMAP_STORAGE char msgfile[PATH_MAX], textfile[PATH_MAX], backup[PATH_MAX], backup_textfile[PATH_MAX]; #endif @@ -6593,7 +6600,8 @@ static int forward_message(struct ast_channel *chan, char *context, struct vm_st } else { /* Ask for an extension */ res = ast_streamfile(chan, "vm-extension", chan->language); /* "extension" */ - if (res) + prompt_played++; + if (res || prompt_played > 4) break; if ((res = ast_readstring(chan, username, sizeof(username) - 1, 2000, 10000, "#") < 0)) break;