Merged revisions 125741 via svnmerge from

https://origsvn.digium.com/svn/asterisk/trunk

................
r125741 | tilghman | 2008-06-27 07:28:38 -0500 (Fri, 27 Jun 2008) | 15 lines

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

........
r125740 | tilghman | 2008-06-27 07:19:39 -0500 (Fri, 27 Jun 2008) | 7 lines

Add proper deadlock avoidance.
(closes issue #12914)
 Reported by: ozan
 Patches: 
       20080625__bug12914.diff.txt uploaded by Corydon76 (license 14)
 Tested by: ozan

........

................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@125742 65c4cc65-6c06-0410-ace0-fbb531ad65f3
1.6.0
Tilghman Lesher 17 years ago
parent ca5d70859b
commit d93ad61286

@ -193,13 +193,13 @@ static int local_queue_frame(struct local_pvt *p, int isoutbound, struct ast_fra
while (other && ast_channel_trylock(other)) { while (other && ast_channel_trylock(other)) {
ast_mutex_unlock(&p->lock); ast_mutex_unlock(&p->lock);
if (us && us_locked) { if (us && us_locked) {
ast_channel_unlock(us); do {
} CHANNEL_DEADLOCK_AVOIDANCE(us);
usleep(1); } while (ast_mutex_trylock(&p->lock));
if (us && us_locked) { } else {
ast_channel_lock(us); usleep(1);
ast_mutex_lock(&p->lock);
} }
ast_mutex_lock(&p->lock);
other = isoutbound ? p->owner : p->chan; other = isoutbound ? p->owner : p->chan;
} }
@ -516,7 +516,12 @@ static int local_hangup(struct ast_channel *ast)
if (!p) if (!p)
return -1; return -1;
ast_mutex_lock(&p->lock); while (ast_mutex_trylock(&p->lock)) {
ast_channel_unlock(ast);
usleep(1);
ast_channel_lock(ast);
}
if (p->chan && ast_test_flag(ast, AST_FLAG_ANSWERED_ELSEWHERE)) if (p->chan && ast_test_flag(ast, AST_FLAG_ANSWERED_ELSEWHERE))
ast_set_flag(p->chan, AST_FLAG_ANSWERED_ELSEWHERE); ast_set_flag(p->chan, AST_FLAG_ANSWERED_ELSEWHERE);
isoutbound = IS_OUTBOUND(ast, p); isoutbound = IS_OUTBOUND(ast, p);

Loading…
Cancel
Save