I am reverting the fix made in revision 168128 (and its upward merges)

after being contacted by Olle Johansson and being shown how this fix is
incorrect. Thanks to Olle for clearing this up for me.

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

diff --git a/channels/chan_sip.c b/channels/chan_sip.c
index ec9220e389bc3a8a6a5afaf05af6af5c84f4954c..5d8ed871f01ca15f5cb4618e9617c55beb7283d6 100644 (file)
--- a/channels/chan_sip.c
+++ b/channels/chan_sip.c
@@ -11332,7 +11332,6 @@ static void handle_request_info(struct sip_pvt *p, struct sip_request *req)
        unsigned int event;
        const char *c = get_header(req, "Content-Type");
 
-       check_via(p, req);
        /* Need to check the media/type */
        if (!strcasecmp(c, "application/dtmf-relay") ||
            !strcasecmp(c, "application/vnd.nortelnetworks.digits")) {
@@ -13581,7 +13580,6 @@ static int handle_request_notify(struct sip_pvt *p, struct sip_request *req, str
        char *eventid = NULL;
        char *sep;
 
-       check_via(p, req);
        if( (sep = strchr(event, ';')) ) {      /* XXX bug here - overwriting string ? */
                *sep++ = '\0';
                eventid = sep;
@@ -13709,7 +13707,7 @@ static int handle_request_options(struct sip_pvt *p, struct sip_request *req)
 {
        int res;
 
-       check_via(p, req);
+
        /* XXX Should we authenticate OPTIONS? XXX */
 
        if (p->lastinvite) {
@@ -14905,7 +14903,6 @@ static int handle_request_refer(struct sip_pvt *p, struct sip_request *req, int
 
        int res = 0;
 
-       check_via(p, req);
        if (ast_test_flag(req, SIP_PKT_DEBUG))
                ast_verbose("Call %s got a SIP call transfer from %s: (REFER)!\n", p->callid, ast_test_flag(&p->flags[0], SIP_OUTGOING) ? "callee" : "caller");
 
@@ -15356,7 +15353,6 @@ static int handle_request_bye(struct sip_pvt *p, struct sip_request *req)
 static int handle_request_message(struct sip_pvt *p, struct sip_request *req)
 {
        if (!ast_test_flag(req, SIP_PKT_IGNORE)) {
-               check_via(p, req);
                if (ast_test_flag(req, SIP_PKT_DEBUG))
                        ast_verbose("Receiving message!\n");
                receive_message(p, req);