]> git.ipfire.org Git - thirdparty/asterisk.git/commitdiff
Don't reject all SUBSCRIBE auth requests
authorTerry Wilson <twilson@digium.com>
Wed, 12 Jan 2011 21:05:02 +0000 (21:05 +0000)
committerTerry Wilson <twilson@digium.com>
Wed, 12 Jan 2011 21:05:02 +0000 (21:05 +0000)
When merging another SUBSCRIBE fix from 1.4, some braces were put in
the wrong place. This patch fixes that.

(closes issue #18597)
Reported by: thsgmbh

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

channels/chan_sip.c

index a65a5891ead2ba137deeaac78c146b148e321eb5..238421eccad75acbfacec7dce99e852fbaea0d94 100644 (file)
@@ -21532,10 +21532,10 @@ static int handle_request_subscribe(struct sip_pvt *p, struct sip_request *req,
                                ast_log(LOG_NOTICE, "Failed to authenticate device %s for SUBSCRIBE\n", get_header(req, "From"));
                                transmit_response_reliable(p, "403 Forbidden", req);
                        }
-               }
 
-               pvt_set_needdestroy(p, "authentication failed");
-               return 0;
+                       pvt_set_needdestroy(p, "authentication failed");
+                       return 0;
+               }
        }
 
        /* At this point, authpeer cannot be NULL. Remember we hold a reference,