From: Russell Bryant <russell@russellbryant.com>
Date: Fri, 7 Dec 2007 02:52:38 +0000 (+0000)
Subject: Merged revisions 91693 via svnmerge from
X-Git-Tag: 1.6.0-beta1~511
X-Git-Url: http://git.asterisk.org/gitweb/?p=asterisk%2Fasterisk.git;a=commitdiff_plain;h=547083e21a5a8459354f6568580ee7f90f8f9a35;ds=sidebyside

Merged revisions 91693 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r91693 | russell | 2007-12-06 20:51:22 -0600 (Thu, 06 Dec 2007) | 2 lines

Don't unlock the dialed_interfaces list until we're done messing with the iterator.

........


git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@91700 65c4cc65-6c06-0410-ace0-fbb531ad65f3
---

diff --git a/apps/app_dial.c b/apps/app_dial.c
index 85fab4b..dbe6424 100644
--- a/apps/app_dial.c
+++ b/apps/app_dial.c
@@ -1391,8 +1391,8 @@ static int dial_exec_full(struct ast_channel *chan, void *data, struct ast_flags
 			strcpy(di->interface, interface);
 			AST_LIST_INSERT_TAIL(dialed_interfaces, di, list);
 		} else if (dialed) {
-			AST_LIST_UNLOCK(dialed_interfaces);
 			ast_log(LOG_WARNING, "Skipping dialing interface '%s' again since it has already been dialed\n", di->interface);
+			AST_LIST_UNLOCK(dialed_interfaces);
 			fulldial++;
 			free(tmp);
 			continue;