[asterisk-bugs] [Asterisk 0013821]: [patch] segfault checking messages using mwi events

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=13821 
====================================================================== 
Reported By:                DEA
Assigned To:                mvanbaak
====================================================================== 
Project:                    Asterisk
Issue ID:                   13821
Category:                   Channels/chan_skinny
Reproducibility:            always
Severity:                   crash
Priority:                   normal
Status:                     assigned
Asterisk Version:           1.6.0.1 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             2008-10-31 23:01 CDT
Last Modified:              2008-12-25 16:43 CST
====================================================================== 
Summary:                    [patch] segfault checking messages using mwi events
Description: 
I have been chasing a regular crash involves chan_skinny and IMAP enable
voicemail.

I'm following the bugs related to the c-client related crashes, which may
help, but the more I looked at the code that I ported from chan_zap to 
chan_skinny to enable mwi events, the more convinced I became that it was
just
not right.

So I dove into chan_sip to get a handle on how mwi event callbacks should
be
used and developed the attached patch.
====================================================================== 

---------------------------------------------------------------------- 
 (0096954) mvanbaak (manager) - 2008-12-25 16:43
 http://bugs.digium.com/view.php?id=13821#c96954 
---------------------------------------------------------------------- 
I'm not sure wether the SKINNY_LAMP_BLINK is a toggle.
Since you are sending this to the phone when the mwiblink is set, no
matter if there's voicemail waiting or not.
I think you just have to send SKINNY_LAMP_OFF in that case instead of the
_BLINK

Other than that the patch looks fine and I'll testrun it once I get home
this weekend. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-12-25 16:43 mvanbaak       Note Added: 0096954                          
======================================================================