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

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

diff --git a/channels/chan_sip.c b/channels/chan_sip.c
index dac377e918bcf6e72d50bb855b6b9d6b821165dc..08f3c9ebf63aab6fcf89aa32e7bdc1575faf1223 100644 (file)
--- a/channels/chan_sip.c
+++ b/channels/chan_sip.c
@@ -23027,6 +23027,8 @@ static void handle_response_invite(struct sip_pvt *p, int resp, const char *rest
                }
                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 @@ static void handle_response_subscribe(struct sip_pvt *p, int resp, const char *r
                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 @@ static int handle_response_register(struct sip_pvt *p, int resp, const char *res
                }
                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 @@ static void handle_response(struct sip_pvt *p, int resp, const char *rest, struc
                                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);