[asterisk-bugs] [Asterisk 0011256]: [patch] Deadlock in find_session(unsigned long ident)

[noreply at bugs.digium.com]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=11256 
====================================================================== 
Reported By:                ys
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   11256
Category:                   Core/ManagerInterface
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     new
Asterisk Version:            SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  1.4  
SVN Revision (number only!): 89283 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             11-15-2007 05:36 CST
Last Modified:              11-15-2007 08:52 CST
====================================================================== 
Summary:                    [patch] Deadlock in find_session(unsigned long
ident)
Description: 

Missed ast_mutex_unlock(&s->__lock); before exit
AST_LIST_TRAVERSE(&sessions, s, list) at break; point.

Found in 1.4 branch, 1.4.13 and trunk.
====================================================================== 

---------------------------------------------------------------------- 
 eliel - 11-15-07 08:52  
---------------------------------------------------------------------- 
ys: I think there is a missing ast_mutex_unlock() there because
find_session() not only finds the session, also locks the returned session. 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
11-15-07 08:52  eliel          Note Added: 0073713                          
======================================================================