[asterisk-commits] russell: branch 1.4 r69579 - /branches/1.4/res/res_features.c

[SVN commits to the Asterisk project]

Author: russell
Date: Fri Jun 15 15:18:58 2007
New Revision: 69579

URL: http://svn.digium.com/view/asterisk?view=rev&rev=69579
Log:
Fix a silly deadlock in res_features that I found while debugging on one of
blitzrage's test machines.  It was one of the situations where he was seeing
hung channels, and may be the cause of some of the reports from other people.
(related to issue #9235)

Modified:
    branches/1.4/res/res_features.c

Modified: branches/1.4/res/res_features.c
URL: http://svn.digium.com/view/asterisk/branches/1.4/res/res_features.c?view=diff&rev=69579&r1=69578&r2=69579
==============================================================================
--- branches/1.4/res/res_features.c (original)
+++ branches/1.4/res/res_features.c Fri Jun 15 15:18:58 2007
@@ -1083,8 +1083,10 @@
 
 	while ((tok = strsep(&tmp, "#"))) {
 		AST_LIST_LOCK(&feature_list);	
-		if (!(feature = find_dynamic_feature(tok)))
+		if (!(feature = find_dynamic_feature(tok))) {
+			AST_LIST_UNLOCK(&feature_list);
 			continue;
+		}
 			
 		/* Feature is up for consideration */
 		if (!strcmp(feature->exten, code)) {