[Home]

Summary:ASTERISK-04585: [patch] The flag "transfertobusy" doesn't work.
Reporter:Thomas (thomas andrews)Labels:
Date Opened:2005-07-14 03:27:44Date Closed:2008-01-15 15:41:31.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) dial_busy.patch
Description:chan_zap.c revision 1.462 was patched similarly to this:

if( p->owner->_state == AST_STATE_BUSY )

Unfortunately however, p->owner->_state never gets set to
AST_STATE_BUSY, thus making these sections of the code inoperable.

This patch changes the state to AST_STATE_BUSY in two places:
1. When the "Busy" built-in function is used, and
2. In app_dial.c where "Everyone is busy/congested"
Comments:By: Kevin P. Fleming (kpfleming) 2005-07-15 11:56:16

Fixed in CVS HEAD, although I did not modify app_dial, since deciding whether the channel is busy or not would then fall to the dialplan writer. I also added AST_STATE_BUSY setting to pbx_builtin_congestion, since it would also apply. Thanks for the detective work!
By: Digium Subversion (svnbot) 2008-01-15 15:41:30.000-0600

Repository: asterisk
Revision: 6136

U   trunk/pbx.c

------------------------------------------------------------------------
r6136 | kpfleming | 2008-01-15 15:41:29 -0600 (Tue, 15 Jan 2008) | 2 lines

ensure that channels that are busy/congested are marked AST_STATE_BUSY (bug ASTERISK-4585)

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

http://svn.digium.com/view/asterisk?view=rev&revision=6136
By: Digium Subversion (svnbot) 2008-01-15 15:41:31.000-0600

Repository: asterisk
Revision: 6137

U   trunk/channels/chan_zap.c

------------------------------------------------------------------------
r6137 | kpfleming | 2008-01-15 15:41:30 -0600 (Tue, 15 Jan 2008) | 2 lines

various code formatting changes and cleanups related to bug ASTERISK-4585

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

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