diff options
| author | twilson <twilson@f38db490-d61c-443f-a65b-d21fe96a405b> | 2011-01-12 21:05:02 +0000 |
|---|---|---|
| committer | twilson <twilson@f38db490-d61c-443f-a65b-d21fe96a405b> | 2011-01-12 21:05:02 +0000 |
| commit | b4d73274745fdfeb75d1ecd0d74c70f182840738 (patch) | |
| tree | b61bcb3d66d6162223b4f682114e3c6cdb818a8b /channels | |
| parent | 07db0e7e6378926e642555999914e336e5f1879d (diff) | |
Don't reject all SUBSCRIBE auth requests
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: http://svn.digium.com/svn/asterisk/branches/1.6.2@301682 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'channels')
| -rw-r--r-- | channels/chan_sip.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/channels/chan_sip.c b/channels/chan_sip.c index a65a5891e..238421ecc 100644 --- a/channels/chan_sip.c +++ b/channels/chan_sip.c @@ -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, |