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

Tue Apr 7 19:02:39 CDT 2009

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/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-04-07 19:02 CDT
====================================================================== 
Summary:                    Regression: http://bugs.digium.com/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 http://bugs.digium.com/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 ==.

====================================================================== 

---------------------------------------------------------------------- 
 (0102874) svnbot (reporter) - 2009-04-07 19:02
 http://bugs.digium.com/view.php?id=14686#c102874 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 186839

_U  branches/1.6.1/
U   branches/1.6.1/channels/chan_sip.c

------------------------------------------------------------------------
r186839 | mmichelson | 2009-04-07 19:02:39 -0500 (Tue, 07 Apr 2009) | 12
lines

Merged revisions 186837 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

........
  r186837 | mmichelson | 2009-04-07 19:01:49 -0500 (Tue, 07 Apr 2009) | 7
lines

  Fix bad merge from fix for issue 13867.

  (closes issue http://bugs.digium.com/view.php?id=14686)
  Reported by: davidw
........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=186839 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-04-07 19:02 svnbot         Checkin                                      
2009-04-07 19:02 svnbot         Note Added: 0102874                          
======================================================================