diff options
| author | kpfleming <kpfleming@f38db490-d61c-443f-a65b-d21fe96a405b> | 2007-09-04 16:41:14 +0000 |
|---|---|---|
| committer | kpfleming <kpfleming@f38db490-d61c-443f-a65b-d21fe96a405b> | 2007-09-04 16:41:14 +0000 |
| commit | b970a6e03d42ab92dc3765db4ad447a144c16c67 (patch) | |
| tree | d8b2dcc4de154dc0ad9e60c8f37c7a68a2269d9f /channels | |
| parent | 9db3e103c31cf2c15d17200b8f4a05a0c6b8cce9 (diff) | |
Merged revisions 81442 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4
........
r81442 | kpfleming | 2007-09-04 11:40:39 -0500 (Tue, 04 Sep 2007) | 2 lines
there is no point in sending 401 Unauthorized to a UAS that sent us a properly-formatted Authentication header with the expected username and nonce but an incorrect response (which indicates the shared secret does not match)... instead, let's send 403 Forbidden so that the UAS doesn't retry with the same authentication credentials repeatedly
........
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@81443 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'channels')
| -rw-r--r-- | channels/chan_sip.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/channels/chan_sip.c b/channels/chan_sip.c index 753f28352..7c8dc000a 100644 --- a/channels/chan_sip.c +++ b/channels/chan_sip.c @@ -9108,11 +9108,11 @@ static enum check_auth_result check_auth(struct sip_pvt *p, struct sip_request * } /* Ok, we have a bad username/secret pair */ - /* Challenge again, and again, and again */ - transmit_response_with_auth(p, response, req, p->randdata, reliable, respheader, 0); - sip_scheddestroy(p, DEFAULT_TRANS_TIMEOUT); + /* Tell the UAS not to re-send this authentication data, because + it will continue to fail + */ - return AUTH_CHALLENGE_SENT; + return AUTH_SECRET_FAILED; } /*! \brief Change onhold state of a peer using a pvt structure */ |