diff options
author | russell <russell@f38db490-d61c-443f-a65b-d21fe96a405b> | 2007-08-13 20:37:50 +0000 |
---|---|---|
committer | russell <russell@f38db490-d61c-443f-a65b-d21fe96a405b> | 2007-08-13 20:37:50 +0000 |
commit | d8dd30cfc56c73911570ec2c336733763b939833 (patch) | |
tree | 5885fc5cbb9334f90752150b41b79225a2e16600 /channels/chan_iax2.c | |
parent | 1757c35048b69ea298995eefd6adced2f5056843 (diff) |
Don't call find_peer in registry_authrequest with the pvt lock held to avoid a
deadlock.
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@79301 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'channels/chan_iax2.c')
-rw-r--r-- | channels/chan_iax2.c | 22 |
1 files changed, 17 insertions, 5 deletions
diff --git a/channels/chan_iax2.c b/channels/chan_iax2.c index 1ad1dc9a9..0b9bd9b80 100644 --- a/channels/chan_iax2.c +++ b/channels/chan_iax2.c @@ -5794,13 +5794,21 @@ static int update_registry(struct sockaddr_in *sin, int callno, char *devtype, i return send_command_final(iaxs[callno], AST_FRAME_IAX, IAX_COMMAND_REGACK, 0, ied.buf, ied.pos, -1); } -static int registry_authrequest(const char *name, int callno) +static int registry_authrequest(int callno) { struct iax_ie_data ied; struct iax2_peer *p; char challenge[10]; + const char *peer_name; + + peer_name = ast_strdupa(iaxs[callno]->peer); + /* SLD: third call to find_peer in registration */ - p = find_peer(name, 1); + ast_mutex_unlock(&iaxsl[callno]); + p = find_peer(peer_name, 1); + ast_mutex_lock(&iaxsl[callno]); + if (!iaxs[callno]) + return -1; if (p) { memset(&ied, 0, sizeof(ied)); iax_ie_append_short(&ied, IAX_IE_AUTHMETHODS, p->authmethods); @@ -5811,12 +5819,12 @@ static int registry_authrequest(const char *name, int callno) /* snprintf(iaxs[callno]->challenge, sizeof(iaxs[callno]->challenge), "%d", (int)ast_random()); */ iax_ie_append_str(&ied, IAX_IE_CHALLENGE, iaxs[callno]->challenge); } - iax_ie_append_str(&ied, IAX_IE_USERNAME, name); + iax_ie_append_str(&ied, IAX_IE_USERNAME, peer_name); if (ast_test_flag(p, IAX_TEMPONLY)) destroy_peer(p); return send_command(iaxs[callno], AST_FRAME_IAX, IAX_COMMAND_REGAUTH, 0, ied.buf, ied.pos, -1);; } - ast_log(LOG_WARNING, "No such peer '%s'\n", name); + ast_log(LOG_WARNING, "No such peer '%s'\n", peer_name); return 0; } @@ -7650,7 +7658,11 @@ retryowner2: } break; } - registry_authrequest(iaxs[fr->callno]->peer, fr->callno); + registry_authrequest(fr->callno); + if (!iaxs[fr->callno]) { + ast_mutex_unlock(&iaxsl[fr->callno]); + return 1; + } break; case IAX_COMMAND_REGACK: if (iax2_ack_registry(&ies, &sin, fr->callno)) |