Merged revisions 104084 via svnmerge from
authorJoshua Colp <jcolp@digium.com>
Mon, 25 Feb 2008 16:18:46 +0000 (16:18 +0000)
committerJoshua Colp <jcolp@digium.com>
Mon, 25 Feb 2008 16:18:46 +0000 (16:18 +0000)
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r104084 | file | 2008-02-25 12:16:13 -0400 (Mon, 25 Feb 2008) | 6 lines

If a resubscription comes in for a dialog we no longer know about tell the remote side that the dialog does not exist so they subscribe again using a new dialog.
(closes issue #10727)
Reported by: s0l4rb03
Patches:
      10727-2.diff uploaded by file (license 11)

........

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

channels/chan_sip.c

index 84a52c4..2507151 100644 (file)
@@ -17392,6 +17392,18 @@ static int handle_request_subscribe(struct sip_pvt *p, struct sip_request *req,
        }
 
        if (!req->ignore && !resubscribe) {     /* Set up dialog, new subscription */
+               const char *to = get_header(req, "To");
+               char totag[128];
+
+               /* Check to see if a tag was provided, if so this is actually a resubscription of a dialog we no longer know about */
+               if (!ast_strlen_zero(to) && gettag(req, "To", totag, sizeof(totag))) {
+                       if (req->debug)
+                               ast_verbose("Received resubscription for a dialog we no longer know about. Telling remote side to subscribe again.\n");
+                       transmit_response(p, "481 Subscription does not exist", req);
+                       p->needdestroy = 1;
+                       return 0;
+               }
+
                /* Use this as the basis */
                if (req->debug)
                        ast_verbose("Creating new subscription\n");