[asterisk-commits] dvossel: branch 1.6.2 r316329 - in /branches/1.6.2: ./ channels/chan_local.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Tue May 3 16:30:00 CDT 2011


Author: dvossel
Date: Tue May  3 16:29:55 2011
New Revision: 316329

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

........
  r316328 | dvossel | 2011-05-03 16:27:59 -0500 (Tue, 03 May 2011) | 10 lines
  
  Fixes chan_local crashs in local_fixup()
  
  Thanks OEJ for tracking down the issue and submitting the patch.
  
  (closes issue #19053)
  Reported by: oej
  Tested by: oej
  
  Review: https://reviewboard.asterisk.org/r/1158/
........

Modified:
    branches/1.6.2/   (props changed)
    branches/1.6.2/channels/chan_local.c

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

Modified: branches/1.6.2/channels/chan_local.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.6.2/channels/chan_local.c?view=diff&rev=316329&r1=316328&r2=316329
==============================================================================
--- branches/1.6.2/channels/chan_local.c (original)
+++ branches/1.6.2/channels/chan_local.c Tue May  3 16:29:55 2011
@@ -526,7 +526,7 @@
 		p->chan = newchan;
 
 	/* Do not let a masquerade cause a Local channel to be bridged to itself! */
-	if (!ast_check_hangup(newchan) && (p->owner->_bridge == p->chan || p->chan->_bridge == p->owner)) {
+	if (!ast_check_hangup(newchan) && ((p->owner && p->owner->_bridge == p->chan) || (p->chan && p->chan->_bridge == p->owner))) {
 		ast_log(LOG_WARNING, "You can not bridge a Local channel to itself!\n");
 		ao2_unlock(p);
 		ast_queue_hangup(newchan);




More information about the asterisk-commits mailing list