Summary:	ASTERISK-14620: [patch] dahdi_read unbalanced ast_mutex_lock and ast_mutex_unlock
Reporter:	Alec Davis (alecdavis)	Labels:
Date Opened:	2009-08-09 04:25:21	Date Closed:	2009-10-06 19:03:07
Priority:	Trivial	Regression?	No
Status:	Closed/Complete	Components:	Channels/chan_dahdi
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:	( 0) dahdi_read.missing_unlock.diff.txt
Description:	if in alarm, the lock isn't removed. if deferred dialing fails, the lock isn't removed.
Comments:	By: Alec Davis (alecdavis) 2009-09-14 15:11:45 low risk: lets get this closed this after committing. By: Digium Subversion (svnbot) 2009-10-06 18:54:34 Repository: asterisk Revision: 222462 U   branches/1.4/channels/chan_dahdi.c ------------------------------------------------------------------------ r222462 | jpeeler | 2009-10-06 18:54:33 -0500 (Tue, 06 Oct 2009) | 8 lines Add missing unlock(s) in dahdi_read (two cases in trunk) (closes issue ASTERISK-14620) Reported by: alecdavis ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=222462 By: Digium Subversion (svnbot) 2009-10-06 18:59:17 Repository: asterisk Revision: 222463 _U  trunk/ U   trunk/channels/chan_dahdi.c ------------------------------------------------------------------------ r222463 | jpeeler | 2009-10-06 18:59:16 -0500 (Tue, 06 Oct 2009) | 14 lines Merged revisions 222462 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........  r222462 | jpeeler | 2009-10-06 18:51:19 -0500 (Tue, 06 Oct 2009) | 8 lines    Add missing unlock(s) in dahdi_read    (two cases in trunk)    (closes issue ASTERISK-14620)  Reported by: alecdavis ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=222463 By: Digium Subversion (svnbot) 2009-10-06 19:00:57 Repository: asterisk Revision: 222464 _U  branches/1.6.0/ U   branches/1.6.0/channels/chan_dahdi.c ------------------------------------------------------------------------ r222464 | jpeeler | 2009-10-06 19:00:57 -0500 (Tue, 06 Oct 2009) | 21 lines Merged revisions 222463 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................  r222463 | jpeeler | 2009-10-06 18:56:01 -0500 (Tue, 06 Oct 2009) | 14 lines    Merged revisions 222462 via svnmerge from  https://origsvn.digium.com/svn/asterisk/branches/1.4    ........    r222462 | jpeeler | 2009-10-06 18:51:19 -0500 (Tue, 06 Oct 2009) | 8 lines        Add missing unlock(s) in dahdi_read        (two cases in trunk)        (closes issue ASTERISK-14620)    Reported by: alecdavis  ........ ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=222464 By: Digium Subversion (svnbot) 2009-10-06 19:01:45 Repository: asterisk Revision: 222465 _U  branches/1.6.1/ U   branches/1.6.1/channels/chan_dahdi.c ------------------------------------------------------------------------ r222465 | jpeeler | 2009-10-06 19:01:44 -0500 (Tue, 06 Oct 2009) | 21 lines Merged revisions 222463 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................  r222463 | jpeeler | 2009-10-06 18:56:01 -0500 (Tue, 06 Oct 2009) | 14 lines    Merged revisions 222462 via svnmerge from  https://origsvn.digium.com/svn/asterisk/branches/1.4    ........    r222462 | jpeeler | 2009-10-06 18:51:19 -0500 (Tue, 06 Oct 2009) | 8 lines        Add missing unlock(s) in dahdi_read        (two cases in trunk)        (closes issue ASTERISK-14620)    Reported by: alecdavis  ........ ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=222465 By: Digium Subversion (svnbot) 2009-10-06 19:03:07 Repository: asterisk Revision: 222466 _U  branches/1.6.2/ U   branches/1.6.2/channels/chan_dahdi.c ------------------------------------------------------------------------ r222466 | jpeeler | 2009-10-06 19:03:06 -0500 (Tue, 06 Oct 2009) | 21 lines Merged revisions 222463 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................  r222463 | jpeeler | 2009-10-06 18:56:01 -0500 (Tue, 06 Oct 2009) | 14 lines    Merged revisions 222462 via svnmerge from  https://origsvn.digium.com/svn/asterisk/branches/1.4    ........    r222462 | jpeeler | 2009-10-06 18:51:19 -0500 (Tue, 06 Oct 2009) | 8 lines        Add missing unlock(s) in dahdi_read        (two cases in trunk, and 1.6.2)        (closes issue ASTERISK-14620)    Reported by: alecdavis  ........ ................ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=222466