[asterisk-commits] russell: branch 1.4 r107161 - /branches/1.4/main/pbx.c

[SVN commits to the Asterisk project]

Author: russell
Date: Mon Mar 10 15:17:11 2008
New Revision: 107161

URL: http://svn.digium.com/view/asterisk?view=rev&rev=107161
Log:
Fix another bug specifically related to asynchronous call origination.  Once the
PBX is started on the channel using ast_pbx_start(), then the ownership of the
channel has been passed on to another thread.  We can no longer access it in this
code.  If the channel gets hung up very quickly, it is possible that we could
access a channel that has been free'd.

(inspired by BE-386)

Modified:
    branches/1.4/main/pbx.c

Modified: branches/1.4/main/pbx.c
URL: http://svn.digium.com/view/asterisk/branches/1.4/main/pbx.c?view=diff&rev=107161&r1=107160&r2=107161
==============================================================================
--- branches/1.4/main/pbx.c (original)
+++ branches/1.4/main/pbx.c Mon Mar 10 15:17:11 2008
@@ -5027,9 +5027,9 @@
 							ast_channel_unlock(chan);
 						}
 						ast_hangup(chan);
-						chan = NULL;
 						res = -1;
 					}
+					chan = NULL;
 				}
 			} else {
 				if (option_verbose > 3)