[asterisk-bugs] [Asterisk 0014686]: Regression: #13867 Reject an incoming call to peer due to call limit with "603 Declined". It`s not correct.

[Asterisk Bug Tracker]

The following issue has been set as RELATED TO issue 0014802. 
====================================================================== 
https://issues.asterisk.org/view.php?id=14686 
====================================================================== 
Reported By:                davidw
Assigned To:                mmichelson
====================================================================== 
Project:                    Asterisk
Issue ID:                   14686
Category:                   Channels/chan_sip/General
Reproducibility:            have not tried
Severity:                   minor
Priority:                   normal
Status:                     closed
Asterisk Version:           1.6.0.6 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2009-03-17 11:42 CDT
Last Modified:              2009-05-22 12:55 CDT
====================================================================== 
Summary:                    Regression:
https://issues.asterisk.org/view.php?id=13867 Reject an incoming call to peer
due to call limit with "603 Declined". It`s not correct.
Description: 
When the patch for https://issues.asterisk.org/view.php?id=13867 got applied to
the 1.6.0 branch the test that
qualifies the setting of the hangup cause seems to have got inverted, so,
in 1.4.0 (SVN 181768) we have:

3079 	res = update_call_counter(p, INC_CALL_RINGING); 
3080 	if ( res != -1 ) {
......
3103 	} else {
3104 		ast->hangupcause = AST_CAUSE_USER_BUSY;
3105 	}
3106 	return res;

In 1.6.0.6 (r178024) we have:
4300 	res = update_call_counter(p, INC_CALL_RINGING);
4301  
4302 	if (res == -1) {
4303 		return res;
4304 	} else {
4305 		ast->hangupcause = AST_CAUSE_USER_BUSY;
4306 	}
 
Note that the != has become an ==.
 


======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
related to          0014802 Cause code for INVITE timeout doesn't m...
====================================================================== 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-05-22 12:55 lmadsen        Relationship added       related to 0014802  
======================================================================