[asterisk-bugs] [Asterisk 0017135]: [patch] Message count incorrect

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=17135 
====================================================================== 
Reported By:                edhorton
Assigned To:                tilghman
====================================================================== 
Project:                    Asterisk
Issue ID:                   17135
Category:                   Applications/app_voicemail/IMAP
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     confirmed
Target Version:             1.6.2.9
Asterisk Version:           1.6.2.6 
JIRA:                       SWP-1300 
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2010-04-03 14:12 CDT
Last Modified:              2010-05-14 12:53 CDT
====================================================================== 
Summary:                    [patch] Message count incorrect
Description: 
Version 1.6.2.5 and Dovecot IMAP Server for Voicemail storage.  All works
perfectly except for message counts displayed on Polycom phones.  New
message count is always 2 times the correct count which is reported
corectly in voicemailmain.  Old message count is always zero, but again,
reported correctly in voicemailmain.  The following sip notify was sent
after a voicemail was left when previously empty.

NOTIFY sip:3001 at 192.168.1.163:5060 SIP/2.0
Via: SIP/2.0/UDP 192.168.1.120:5060;branch=z9hG4bK6fc1522a;rport
Max-Forwards: 70
From: "asterisk" <sip:asterisk at 192.168.1.120>;tag=as3bbf24be
To: <sip:3001 at 192.168.1.163:5060>
Contact: <sip:asterisk at 192.168.1.120>
Call-ID: 0b0abc554d1721e6231d8257714dd021 at 192.168.1.120
CSeq: 102 NOTIFY
User-Agent: Asterisk PBX 1.6.2.5
Event: message-summary
Content-Type: application/simple-message-summary
Content-Length: 94

Messages-Waiting: yes
Message-Account: sip:asterisk at 192.168.1.120
Voice-Message: 2/0 (0/0)

======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
related to          0017306 [patch] Sequencing numbers of IMAP emai...
related to          0016350 Deleting Multiple IMAP voicemails does ...
====================================================================== 

---------------------------------------------------------------------- 
 (0121918) ebroad (manager) - 2010-05-14 12:53
 https://issues.asterisk.org/view.php?id=17135#c121918 
---------------------------------------------------------------------- 
The attached patch is a better version of part of the patch that is
attached to 17306, but the code in both accomplishes the same thing. The
problem is twofold:

A. folder_int() is missing a case for the "Urgent" folder, so INBOX
actually gets counted twice (the attached patch does not contain this
part)
B. pgm->flagged, unflagged, seen and unseen may be uninitialized, which
could lead to regular and urgent messages being counted twice, the attached
patch by tilghman explicitly sets these flags depending on which folder and
what type of messages are being requested. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-05-14 12:53 ebroad         Note Added: 0121918                          
======================================================================