[asterisk-bugs] [Asterisk 0016320]: [patch] Incorrect reloading of realtime peer causes mailbox list to expand indefinitely

[Asterisk Bug Tracker]

The following issue is now READY FOR REVIEW. 
====================================================================== 
https://issues.asterisk.org/view.php?id=16320 
====================================================================== 
Reported By:                Marquis
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   16320
Category:                   Channels/chan_sip/General
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     ready for review
Asterisk Version:           1.6.1.10 
JIRA:                       SWP-1352 
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): 1.6.1 
SVN Revision (number only!): 231169 
Request Review:              
====================================================================== 
Date Submitted:             2009-11-24 18:22 CST
Last Modified:              2010-05-18 12:46 CDT
====================================================================== 
Summary:                    [patch] Incorrect reloading of realtime peer causes
mailbox list to expand indefinitely
Description: 
Due to the current code's inability to short-circuit the reloading of a
realtime peer when it's looking for a user, the mailbox list is repeatedly
appended to.

For example, if you issue a 'sip show peer 1234', you may get something
like:
Mailbox     : 1234 at default

But after any inbound peer match (resulting in a call to find_peer), it
will result in:
Mailbox     : 1234 at default, 1234 at default

And so on, ad infinitum.

This can cause a large number of NOTIFYs to be sent to the peer, in some
cases resulting in a phone crash or reboot.

There may be other effects, but this is the observed behavior.
======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
related to          0016021 [patch] sippeers loaded with realtime a...
has duplicate       0017254 [patch] Multiple SIP NOTIFY packets sen...
related to          0017313 SIP NOTIFY bombardment after hangup voi...
====================================================================== 

---------------------------------------------------------------------- 
 (0122063) lmadsen (administrator) - 2010-05-18 12:46
 https://issues.asterisk.org/view.php?id=16320#c122063 
---------------------------------------------------------------------- 
Marking this issue as Ready for Review. It'd be nice if we could get this
up on reviewboard and someone to sign off on it before we commit,
especially since this is a change to chan_sip.c. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-05-18 12:46 lmadsen        Note Added: 0122063                          
2010-05-18 12:46 lmadsen        Status                   ready for testing =>
ready for review
======================================================================