Author: mjordan Date: Wed Oct 29 20:58:02 2014 New Revision: 426600 URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=426600 Log: channels/chan_sip: Add improved support for 4xx error codes
This patch adds support for 414, 493, 479, and a stray 400 response in REGISTER response handling. This helps interoperability in a number of scenarios. Review: https://reviewboard.asterisk.org/r/3437 patches: rb3437.patch uploaded by oej (License 5267) ........ Merged revisions 426599 from http://svn.asterisk.org/svn/asterisk/branches/1.8 Modified: branches/11/ (props changed) branches/11/channels/chan_sip.c Propchange: branches/11/ ------------------------------------------------------------------------------ Binary property 'branch-1.8-merged' - no diff available. Modified: branches/11/channels/chan_sip.c URL: http://svnview.digium.com/svn/asterisk/branches/11/channels/chan_sip.c?view=diff&rev=426600&r1=426599&r2=426600 ============================================================================== --- branches/11/channels/chan_sip.c (original) +++ branches/11/channels/chan_sip.c Wed Oct 29 20:58:02 2014 @@ -23027,6 +23027,8 @@ } break; + case 414: /* Bad request URI */ + case 493: /* Undecipherable */ case 404: /* Not found */ xmitres = transmit_request(p, SIP_ACK, seqno, XMIT_UNRELIABLE, FALSE); if (p->owner && !req->ignore) { @@ -23258,12 +23260,16 @@ ASTOBJ_UNREF(p->mwi, sip_subscribe_mwi_destroy); pvt_set_needdestroy(p, "received 481 response"); break; + + case 400: /* Bad Request */ + case 414: /* Request URI too long */ + case 493: /* Undecipherable */ case 500: case 501: ast_log(LOG_WARNING, "Subscription failed for MWI. The remote side may have suffered a heart attack.\n"); p->mwi->call = NULL; ASTOBJ_UNREF(p->mwi, sip_subscribe_mwi_destroy); - pvt_set_needdestroy(p, "received 500/501 response"); + pvt_set_needdestroy(p, "received serious error (500/501/493/414/400) response"); break; } } @@ -23437,11 +23443,14 @@ } manager_event(EVENT_FLAG_SYSTEM, "Registry", "ChannelType: SIP\r\nUsername: %s\r\nDomain: %s\r\nStatus: %s\r\n", r->username, r->hostname, regstate2str(r->regstate)); break; - case 479: /* SER: Not able to process the URI - address is wrong in register*/ - ast_log(LOG_WARNING, "Got error 479 on register to %s@%s, giving up (check config)\n", p->registry->username, p->registry->hostname); - pvt_set_needdestroy(p, "received 479 response"); + case 400: /* Bad request */ + case 414: /* Request URI too long */ + case 493: /* Undecipherable */ + case 479: /* Kamailio/OpenSIPS: Not able to process the URI - address is wrong in register*/ + ast_log(LOG_WARNING, "Got error %d on register to %s@%s, giving up (check config)\n", resp, p->registry->username, p->registry->hostname); + pvt_set_needdestroy(p, "received 4xx response"); if (r->call) - r->call = dialog_unref(r->call, "unsetting registry->call pointer-- case 479"); + r->call = dialog_unref(r->call, "unsetting registry->call pointer-- case 4xx"); r->regstate = REG_STATE_REJECTED; AST_SCHED_DEL_UNREF(sched, r->timeout, registry_unref(r, "reg ptr unref from handle_response_register 479")); break; @@ -23906,6 +23915,9 @@ pvt_set_needdestroy(p, "received 403 response"); } break; + case 400: /* Bad Request */ + case 414: /* Request URI too long */ + case 493: /* Undecipherable */ case 404: /* Not found */ if (p->registry && sipmethod == SIP_REGISTER) handle_response_register(p, resp, rest, req, seqno); -- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- svn-commits mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/svn-commits
