[asterisk-bugs] [Asterisk 0011566]: chan_mobile.c has no mutexes for data accessed by different threads

[noreply at bugs.digium.com]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=11566 
====================================================================== 
Reported By:                non-poster
Assigned To:                dbowerman
====================================================================== 
Project:                    Asterisk
Issue ID:                   11566
Category:                   Addons/chan_mobile
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     feedback
Asterisk Version:            SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             12-15-2007 23:21 CST
Last Modified:              01-11-2008 05:26 CST
====================================================================== 
Summary:                    chan_mobile.c has no mutexes for data accessed by
different threads
Description: 
There are no mutexes in chan_mobile.c!  Multiple threads access the same
data, eg the data in the mbl_pvt struct in the devices list.  (To compare,
almost all of the "standard" channel modules use mutexes.)
====================================================================== 

---------------------------------------------------------------------- 
 latysheff - 01-11-08 05:26  
---------------------------------------------------------------------- 
Perhaps, this makes impossible recieving sms during voice call. Even if I
patch code so that I restore channel state after reading SMS, call is hang
up because on finished reading SMS you create channel with the same pvt
struct. 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
01-11-08 05:26  latysheff      Note Added: 0076705                          
======================================================================