Summary: | ASTERISK-04184: [patch] Voicemail Duration Incorrect | ||
Reporter: | ssljivic (ssljivic) | Labels: | |
Date Opened: | 2005-05-16 07:22:57 | Date Closed: | 2008-01-15 15:35:02.000-0600 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | Applications/app_voicemail |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ( 0) vm-duration-issue-1.0.patch | |
Description: | Hi, I have found one more bug in the app_voicemail. When someone perform administration of the VM box while there are active callers that are leaving the messages, a duration can be written to the wrong message status file. I have uploaded the short patch that fixes this. My disclaimer is on file. Regards, Stojan Sljivic ****** STEPS TO REPRODUCE ****** To reproduce this bug, do the following: 1. Leave 2 voicemail messages: msg_A and msg_B(we assume that the mailbox is empty) 2. Move the msg_B to the Old folder 3. Start leaving the third message msg_C 4. While leaving msg_C, copy the msg_B from the Old folder to the INBOX 5. While leaving the msg_C, delete msg_A 6. End msg_C. 7. app_voicemail will write the duration to the wrong message summary TXT file, resulting in msg_C->duration = Unknown and msg_B->duration = <duration of the msg_C> | ||
Comments: | By: ssljivic (ssljivic) 2005-05-16 07:24:37 I'm not sure how I set severity to major. Please change it to minor. Regards, Stojan Sljivic By: Kevin P. Fleming (kpfleming) 2005-05-16 09:31:51 Yeah, I noticed that weird code when reviewing your other patch. Thanks for the fix, committed to CVS HEAD! By: Digium Subversion (svnbot) 2008-01-15 15:35:02.000-0600 Repository: asterisk Revision: 5698 U trunk/apps/app_voicemail.c ------------------------------------------------------------------------ r5698 | kpfleming | 2008-01-15 15:35:02 -0600 (Tue, 15 Jan 2008) | 2 lines clean up message duration writing code (bug ASTERISK-4184) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=5698 |