[asterisk-bugs] [Asterisk 0018486]: [patch] Voicemail files out of sequence

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=18486 
====================================================================== 
Reported By:                bluefox
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   18486
Category:                   Applications/app_voicemail
Reproducibility:            random
Severity:                   minor
Priority:                   normal
Status:                     acknowledged
Asterisk Version:           1.6.2.16-rc1 
JIRA:                       SWP-2755 
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2010-12-16 09:50 CST
Last Modified:              2011-01-19 23:41 CST
====================================================================== 
Summary:                    [patch] Voicemail files out of sequence
Description: 
I get this regularly for some mailboxes. Can't find what they do different
than others, but it's always the same people.  I have no third-party
voicemail tools that is removing voicemails or anything else.

CLI message:
[Dec 16 10:45:43] NOTICE[5587]: app_voicemail.c:7432 open_mailbox:
Mailbox: /var/spool/asterisk/voicemail/xxxx/yyyy/INBOX, expected 0 but
found 3 message(s) in box with max threshold of 100.

In the folder, I find that messages are out of sequence.  Most times
msg000 is missing but msg0001 is there.  Other times msg0000 is there but
(let's say) msg0004 is missing so there is msg0001, 2, 3, 5 and 6.



======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
related to          0018498 [patch] [regression] Resequencing of ma...
====================================================================== 

---------------------------------------------------------------------- 
 (0130759) bluefox (reporter) - 2011-01-19 23:41
 https://issues.asterisk.org/view.php?id=18486#c130759 
---------------------------------------------------------------------- 
Found the issue!!! 

In the function copy_message() of app_voicemail.c (the one for file
storage, not IMAP, around line 4989) you call vm_delete(topath) while it
should be vm_delete(frompath).  You gotta delete the old message, not the
one that was just put in the new mailbox.

I tested it, it seems to work seem.  the sox problem was related to
this,but is gone now. (was trying to convert a non-existent file I guess).

Am I correct? 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2011-01-19 23:41 bluefox        Note Added: 0130759                          
======================================================================