[asterisk-commits] mmichelson: trunk r208229 - /trunk/channels/chan_sip.c

[SVN commits to the Asterisk project]

Author: mmichelson
Date: Thu Jul 23 09:46:53 2009
New Revision: 208229

URL: http://svn.asterisk.org/svn-view/asterisk?view=rev&rev=208229
Log:
Fix potential crash if p->owner is NULL.

Problem was observed when a call-forwarding loop was accidentally
configured.

ABE-1906


Modified:
    trunk/channels/chan_sip.c

Modified: trunk/channels/chan_sip.c
URL: http://svn.asterisk.org/svn-view/asterisk/trunk/channels/chan_sip.c?view=diff&rev=208229&r1=208228&r2=208229
==============================================================================
--- trunk/channels/chan_sip.c (original)
+++ trunk/channels/chan_sip.c Thu Jul 23 09:46:53 2009
@@ -18869,11 +18869,11 @@
 				case 301: /* Moved permanently */
 				case 302: /* Moved temporarily */
 				case 305: /* Use Proxy */
-					{
+				if (p->owner) {
 					struct ast_party_redirecting redirecting = {{0,},};
 					change_redirecting_information(p, req, &redirecting, TRUE);
 					ast_channel_set_redirecting(p->owner, &redirecting);
-					}
+				}
 					/* Fall through */
 				case 486: /* Busy here */
 				case 600: /* Busy everywhere */