[asterisk-bugs] [Asterisk 0012214]: [patch] Added event based MWI to chan_skinny

[noreply at bugs.digium.com]

The following issue has been ASSIGNED. 
====================================================================== 
http://bugs.digium.com/view.php?id=12214 
====================================================================== 
Reported By:                DEA
Assigned To:                mvanbaak
====================================================================== 
Project:                    Asterisk
Issue ID:                   12214
Category:                   Channels/chan_skinny
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     assigned
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 108392 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             03-14-2008 10:59 CDT
Last Modified:              04-27-2008 10:12 CDT
====================================================================== 
Summary:                    [patch] Added event based MWI to chan_skinny
Description: 
Modelled on chan_zap and chan_mgcp.  I have been chasing a segfault
that initially appeared to be related to IMAP and voicemail.  After
looking at way too many core dumps I finally noticed that while the
crash occured in the c-client code, the call trace always pointed
to starting in the do_housekeeping finction in chan_skinny.

The function do_housekeeping calls has_voicemail which in chan_skinny
is just a call to ast_app_has_voicemail.  I reworked the do_housekeeping
and has_voicemail functions to resemble their counterparts in chan_zap
and chan_mgcp.

So far I have not been able to recreate the segfault with light VM
interaction, were as before the patch one call would typically trigger
the dump.
====================================================================== 

---------------------------------------------------------------------- 
 svnbot - 04-27-08 10:12  
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 114700

U   trunk/channels/chan_skinny.c

------------------------------------------------------------------------
r114700 | mvanbaak | 2008-04-27 10:12:01 -0500 (Sun, 27 Apr 2008) | 8
lines

Make MWI in chan_skinny event based modeled after chan_zap and chan_mgcp.

(closes issue http://bugs.digium.com/view.php?id=12214)
Reported by: DEA
Patches:
      chan_skinny-vm-events-v3.txt uploaded by DEA (license 3)
	  Tested by: DEA and me

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=114700 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
04-27-08 10:12  svnbot         Note Added: 0086067                          
04-27-08 10:12  svnbot         Assigned To              qwell => mvanbaak   
======================================================================