Summary:	ASTERISK-11506: Messages silently deleted when user goes over maxmsg in Old folder
Reporter:	Matt Selsky (selsky)	Labels:
Date Opened:	2008-02-25 14:08:37.000-0600	Date Closed:	2008-02-25 17:19:20.000-0600
Priority:	Major	Regression?	No
Status:	Closed/Complete	Components:	Applications/app_voicemail
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:
Description:	We are experiencing a problem with messages being silently deleted without asterisk generating any error message (audio or in logs). To recreate the problem: In /etc/asterisk/voicemail.conf        maxmsg=3 Call and leave 3 voicemail messages for voicmailbox A. With 3 messages in voicemailbox A, caller B get an error recording that mailbox is full. Owner of voicemailbox A calls to listen to the voicemail messages and saves all 3 in Old messages folder. Caller B calls voicemailbox A and this time caller is able to leave a message which is saved in INBOX folder. Owner of voicemailbox A calls to listen to the message, hears the message and hangs up without explicitly saving the message. Asterisk deletes the message from the Inbox, but it is not saved in Old messages folder. (New message is lost) Voicemail should generate some error and should not delete the message from the Inbox.
Comments:	By: Digium Subversion (svnbot) 2008-02-25 15:28:09.000-0600 Repository: asterisk Revision: 104094 U   branches/1.4/apps/app_voicemail.c ------------------------------------------------------------------------ r104094 | tilghman | 2008-02-25 15:28:08 -0600 (Mon, 25 Feb 2008) | 5 lines If the destination folder is full, don't delete a message when exiting. (closes issue ASTERISK-11506) Reported by: selsky Patch by: (myself) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=104094 By: Digium Subversion (svnbot) 2008-02-25 15:49:57.000-0600 Repository: asterisk Revision: 104097 _U  trunk/ U   trunk/apps/app_voicemail.c ------------------------------------------------------------------------ r104097 | tilghman | 2008-02-25 15:49:57 -0600 (Mon, 25 Feb 2008) | 13 lines Merged revisions 104094 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r104094 | tilghman | 2008-02-25 15:31:47 -0600 (Mon, 25 Feb 2008) | 5 lines If the destination folder is full, don't delete a message when exiting. (closes issue ASTERISK-11506) Reported by: selsky Patch by: (myself) ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=104097 By: Digium Subversion (svnbot) 2008-02-25 17:19:20.000-0600 Repository: asterisk Revision: 104105 _U  team/murf/bug11210/ U   team/murf/bug11210/apps/app_voicemail.c U   team/murf/bug11210/channels/chan_agent.c U   team/murf/bug11210/channels/chan_iax2.c U   team/murf/bug11210/channels/chan_sip.c U   team/murf/bug11210/configs/sip.conf.sample U   team/murf/bug11210/doc/siptls.txt U   team/murf/bug11210/funcs/func_global.c U   team/murf/bug11210/main/config.c ------------------------------------------------------------------------ r104105 | murf | 2008-02-25 17:19:19 -0600 (Mon, 25 Feb 2008) | 133 lines Merged revisions 104081,104083,104085,104087-104089,104093,104096-104098 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r104081 | file | 2008-02-25 08:12:48 -0700 (Mon, 25 Feb 2008) | 2 lines Fix building of trunk. dbpass is always going to exist. ................ r104083 | file | 2008-02-25 08:19:58 -0700 (Mon, 25 Feb 2008) | 14 lines Merged revisions 104082 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r104082 | file | 2008-02-25 11:17:18 -0400 (Mon, 25 Feb 2008) | 6 lines Due to recent changes tag will no longer be NULL if not present so we have to use ast_strlen_zero to see if it's actually blank. (closes issue ASTERISK-11502) Reported by: flefoll Patches:      chan_sip.c.br14.patch_pedantic_no_totag uploaded by flefoll (license 244) ........ ................ r104085 | file | 2008-02-25 09:18:46 -0700 (Mon, 25 Feb 2008) | 14 lines Merged revisions 104084 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r104084 | file | 2008-02-25 12:16:13 -0400 (Mon, 25 Feb 2008) | 6 lines If a resubscription comes in for a dialog we no longer know about tell the remote side that the dialog does not exist so they subscribe again using a new dialog. (closes issue ASTERISK-10305) Reported by: s0l4rb03 Patches:      10727-2.diff uploaded by file (license 11) ........ ................ r104087 | russell | 2008-02-25 11:38:51 -0700 (Mon, 25 Feb 2008) | 12 lines Merged revisions 104086 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r104086 | russell | 2008-02-25 12:38:10 -0600 (Mon, 25 Feb 2008) | 4 lines Ensure that the channel doesn't disappear in agent_logoff().  If it does, it could cause a crash. (fixes the crash reported in BE-396) ........ ................ r104088 | bbryant | 2008-02-25 12:00:16 -0700 (Mon, 25 Feb 2008) | 1 line Adding more tls configuration details to sip.conf sample, with a list of valid ciphers provided in both files. .. First commit since July, woot ................ r104089 | file | 2008-02-25 12:02:33 -0700 (Mon, 25 Feb 2008) | 2 lines Instead of outputting a verbose message every so often let's make it a debug message. ................ r104093 | qwell | 2008-02-25 13:50:57 -0700 (Mon, 25 Feb 2008) | 19 lines Merged revisions 104092 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r104092 | qwell | 2008-02-25 14:49:42 -0600 (Mon, 25 Feb 2008) | 11 lines Allow the use of #include and #exec in situations where the max include depth was only 1. Specifically, this fixes using #include and #exec in extconfig.conf. This was basically caused because the config file itself raises the include level to 1. I opted not to raise the include limit, because recursion here could cause very bizarre behavior. Pointed out, and tested by jmls (closes issue ASTERISK-11505) ........ ................ r104096 | file | 2008-02-25 14:40:30 -0700 (Mon, 25 Feb 2008) | 14 lines Merged revisions 104095 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r104095 | file | 2008-02-25 17:37:20 -0400 (Mon, 25 Feb 2008) | 6 lines Make it so a users.conf user creates both a SIP peer and a SIP user. The user will be used for inbound authentication for the device, and peer will be used for placing calls to the device. (closes issue ASTERISK-8785) Reported by: queuetue Patches:      sip-gui-friend.diff uploaded by qwell (license 4) ........ ................ r104097 | tilghman | 2008-02-25 14:53:36 -0700 (Mon, 25 Feb 2008) | 13 lines Merged revisions 104094 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r104094 | tilghman | 2008-02-25 15:31:47 -0600 (Mon, 25 Feb 2008) | 5 lines If the destination folder is full, don't delete a message when exiting. (closes issue ASTERISK-11506) Reported by: selsky Patch by: (myself) ........ ................ r104098 | tilghman | 2008-02-25 14:56:19 -0700 (Mon, 25 Feb 2008) | 7 lines Shared space for variables (instead of letting other channels muck with your own) (closes issue ASTERISK-11392) Reported by: ramonpeek Patches:       20080208__bug11943__2.diff.txt uploaded by Corydon76 (license 14) Tested by: jmls ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=104105