Skip to content
Snippets Groups Projects
Commit 84d0b95d authored by Tilghman Lesher's avatar Tilghman Lesher
Browse files

Merged revisions 257467 via svnmerge from

https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r257467 | tilghman | 2010-04-15 15:24:50 -0500 (Thu, 15 Apr 2010) | 13 lines
  
  Don't recreate peer, when responding to a repeated deregistration attempt.
  
  When a reply to a deregistration is lost in transmit, the client retries the
  deregistration.  Previously, this would cause a realtime/autocreate peer to be
  loaded back into memory, after it had already been correctly purged.  Instead,
  we just want to resend the reply without loading the peer.
  
  (closes issue #16908)
   Reported by: kkm
   Patches: 
         20100412__issue16908.diff.txt uploaded by tilghman (license 14)
   Tested by: kkm
........


git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@257493 65c4cc65-6c06-0410-ace0-fbb531ad65f3
parent bef07ecc
Branches
Tags
No related merge requests found
......@@ -12947,6 +12947,21 @@ static enum check_auth_result register_verify(struct sip_pvt *p, struct sockaddr
 
ast_string_field_set(p, exten, name);
build_contact(p);
if (req->ignore) {
/* Expires is a special case, where we only want to load the peer if this isn't a deregistration attempt */
const char *expires = get_header(req, "Expires");
int expire = atoi(expires);
if (ast_strlen_zero(expires)) { /* No expires header; look in Contact */
if ((expires = strcasestr(get_header(req, "Contact"), ";expires="))) {
expire = atoi(expires + 9);
}
}
if (!ast_strlen_zero(expires) && expire == 0) {
transmit_response_with_date(p, "200 OK", req);
return 0;
}
}
peer = find_peer(name, NULL, TRUE, FINDPEERS, FALSE, 0);
if (!(peer && ast_apply_ha(peer->ha, sin))) {
/* Peer fails ACL check */
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment