[asterisk-commits] rmudgett: branch 1.6.2 r304006 - in /branches/1.6.2: ./ main/features.c

[SVN commits to the Asterisk project]

Author: rmudgett
Date: Tue Jan 25 17:25:32 2011
New Revision: 304006

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=304006
Log:
Merged revisions 304005 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r304005 | rmudgett | 2011-01-25 17:21:09 -0600 (Tue, 25 Jan 2011) | 8 lines
  
  DTMF attended transfers sometimes fail for no apparent reason.
  
  The loop in feature_request_and_dial() can exit when Party C has answered
  without processing an AST_CONTROL_ANSWER.  Also sometimes an
  AST_CONTROL_ANSWER never happens even though Party C has answered.
  
  Don't hangup Party C if he is up or we receive an AST_CONTROL_ANSWER.
........

Modified:
    branches/1.6.2/   (props changed)
    branches/1.6.2/main/features.c

Propchange: branches/1.6.2/
------------------------------------------------------------------------------
Binary property 'branch-1.4-merged' - no diff available.

Modified: branches/1.6.2/main/features.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.6.2/main/features.c?view=diff&rev=304006&r1=304005&r2=304006
==============================================================================
--- branches/1.6.2/main/features.c (original)
+++ branches/1.6.2/main/features.c Tue Jan 25 17:25:32 2011
@@ -2663,9 +2663,8 @@
 
 done:
 	ast_indicate(caller, -1);
-	if (chan && ready) {
-		if (chan->_state == AST_STATE_UP)
-			state = AST_CONTROL_ANSWER;
+	if (chan && (ready || chan->_state == AST_STATE_UP)) {
+		state = AST_CONTROL_ANSWER;
 	} else if (chan) {
 		ast_hangup(chan);
 		chan = NULL;