[asterisk-commits] dvossel: trunk r238492 - /trunk/main/channel.c

[SVN commits to the Asterisk project]

Author: dvossel
Date: Thu Jan  7 14:42:27 2010
New Revision: 238492

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=238492
Log:

fixes ast_transfer stall until hangup if called with a channel that doesn't support transfers

ast_transfer sets res to 0 if there is no technology transfer function,
but then tests for it to be negative before deciding to do an early exit.
As a result, it will will wait for an AST_CONTROL_TRANSFER message that
will never come.

(closes issue #16424)
Reported by: davidw
Patches:
      Issue_16424_trunk_234134.patch uploaded by davidw (license 780)


Modified:
    trunk/main/channel.c

Modified: trunk/main/channel.c
URL: http://svnview.digium.com/svn/asterisk/trunk/main/channel.c?view=diff&rev=238492&r1=238491&r2=238492
==============================================================================
--- trunk/main/channel.c (original)
+++ trunk/main/channel.c Thu Jan  7 14:42:27 2010
@@ -4597,7 +4597,7 @@
 	}
 	ast_channel_unlock(chan);
 
-	if (res < 0) {
+	if (res <= 0) {
 		return res;
 	}