[asterisk-bugs] [Asterisk 0018342]: [patch] chan_sip continues to call add_peer_mwi_subs which continues to increament MWI NOTIFY's

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=18342 
====================================================================== 
Reported By:                nivek
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   18342
Category:                   Channels/chan_sip/Subscriptions
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     new
Asterisk Version:           1.8.0 
JIRA:                        
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2010-11-19 14:57 CST
Last Modified:              2010-11-22 12:01 CST
====================================================================== 
Summary:                    [patch] chan_sip continues to call add_peer_mwi_subs
which continues to increament MWI NOTIFY's
Description: 
Just calling any phone number increments the subscriptions to that phone. 
This is causing random reboots to Polycom (which we use) phones due to the
amount of NOTIFY's sent.  This is on realtime database.
Restarting asterisk resets the counters or whatever is does.  And problem
goes away until it happens again.
====================================================================== 

---------------------------------------------------------------------- 
 (0129025) nivek (reporter) - 2010-11-22 12:01
 https://issues.asterisk.org/view.php?id=18342#c129025 
---------------------------------------------------------------------- 
In looking at issue 16320 again, it seems that Marquis is apparently a GD
prophet.

Quoted by Marquis from issue 16320....
I haven't tested the patch with your new approach, but it looks reasonable
to me. The only reservation I have is that the simpler approach treats the
*symptom* while my original patch fixes the root cause.

The only reason this might be materially different is if there are other,
unknown issues with the way realtime peers are being loaded. I have
absolutely ZERO proof (or even idle speculation, based on a cursory glance
over the code) that this is the case, and so I'm happy with the new way.
chan_sip is a wooly place... 




Funny stuff........ 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-11-22 12:01 nivek          Note Added: 0129025                          
======================================================================