[asterisk-bugs] [Asterisk 0017188]: lack of locking in dahdi_request()

[Asterisk Bug Tracker]

The following issue requires your FEEDBACK. 
====================================================================== 
https://issues.asterisk.org/view.php?id=17188 
====================================================================== 
Reported By:                gknispel_proformatique
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   17188
Category:                   Channels/chan_dahdi
Reproducibility:            have not tried
Severity:                   minor
Priority:                   normal
Status:                     feedback
Asterisk Version:           SVN 
JIRA:                        
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases):  1.4  
SVN Revision (number only!): 257341 
Request Review:              
====================================================================== 
Date Submitted:             2010-04-15 09:05 CDT
Last Modified:              2010-04-15 10:14 CDT
====================================================================== 
Summary:                    lack of locking in dahdi_request()
Description: 
In dahdi_request(), iflist / ifend are acceded without any lock taken -- or
alternatively pri->crvs / pri->crvend are.

I don't know if/when those can be modified, especially regarding crv. For
iflist/ifend, maybe reloads are concerned, and also pseudo channels, which
I don't know what they are used for.
====================================================================== 

---------------------------------------------------------------------- 
 (0120458) lmadsen (administrator) - 2010-04-15 10:14
 https://issues.asterisk.org/view.php?id=17188#c120458 
---------------------------------------------------------------------- 
I'm not quite sure what the issue is here. Is this something that is
causing a problem, or is this just something you saw while doing some code
review? 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-04-15 10:14 lmadsen        Note Added: 0120458                          
2010-04-15 10:14 lmadsen        Status                   new => feedback     
======================================================================