diff options
author | oej <oej@f38db490-d61c-443f-a65b-d21fe96a405b> | 2006-06-08 20:14:54 +0000 |
---|---|---|
committer | oej <oej@f38db490-d61c-443f-a65b-d21fe96a405b> | 2006-06-08 20:14:54 +0000 |
commit | 4ab7d1f74ee039c9b4d1b7edcd578f54f9b3ad61 (patch) | |
tree | 6a3668b51db2b9b44463a17f8eddcbba569a0a26 /channels | |
parent | c5a499f0f22ad8cab9eac3f5ae2225b5b54422ae (diff) |
At least reply if we get a fatal error in handling a packet.
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@33111 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'channels')
-rw-r--r-- | channels/chan_sip.c | 18 |
1 files changed, 11 insertions, 7 deletions
diff --git a/channels/chan_sip.c b/channels/chan_sip.c index 5e8317341..ae8d14f44 100644 --- a/channels/chan_sip.c +++ b/channels/chan_sip.c @@ -13234,7 +13234,7 @@ retrylock: /* becaues this is deadlock-prone, we need to try and unlock if failed */ if (p->owner && ast_channel_trylock(p->owner)) { if (option_debug) - ast_log(LOG_DEBUG, "Failed to grab lock, trying again...\n"); + ast_log(LOG_DEBUG, "Failed to grab owner channel lock, trying again. (SIP call %s)\n", p->callid); ast_mutex_unlock(&p->lock); ast_mutex_unlock(&netlock); /* Sleep for a very short amount of time */ @@ -13242,15 +13242,19 @@ retrylock: if (--lockretry) goto retrylock; } - if (!lockretry) { - ast_log(LOG_ERROR, "We could NOT get the channel lock for %s! \n", p->owner->name); - ast_log(LOG_ERROR, "SIP MESSAGE JUST IGNORED: %s \n", req.data); - ast_log(LOG_ERROR, "BAD! BAD! BAD!\n"); - return 1; - } p->recv = sin; + if (recordhistory) /* This is a request or response, note what it was for */ append_history(p, "Rx", "%s / %s / %s", req.data, get_header(&req, "CSeq"), req.rlPart2); + + if (!lockretry) { + ast_log(LOG_ERROR, "We could NOT get the channel lock for %s! \n", p->owner->name ? p->owner->name : "- no channel name ??? - "); + ast_log(LOG_ERROR, "SIP transaction failed: %s \n", p->callid); + transmit_response(p, "503 Server error", &req); /* We must respond according to RFC 3261 sec 12.2 */ + /* XXX We could add retry-after to make sure they come back */ + append_history(p, "LockFail", "Owner lock failed, transaction failed."); + return 1; + } nounlock = 0; if (handle_request(p, &req, &sin, &recount, &nounlock) == -1) { /* Request failed */ |