[asterisk-commits] file: branch 1.2 r66764 - /branches/1.2/channels/chan_sip.c

[asterisk-commits at lists.digium.com]

Author: file
Date: Thu May 31 11:12:39 2007
New Revision: 66764

URL: http://svn.digium.com/view/asterisk?view=rev&rev=66764
Log:
It is now possible for this path of execution to have the frame pointer be NULL, therefore we need to check for it before trying to access it. (issue #9836 reported by barthpbx)

Modified:
    branches/1.2/channels/chan_sip.c

Modified: branches/1.2/channels/chan_sip.c
URL: http://svn.digium.com/view/asterisk/branches/1.2/channels/chan_sip.c?view=diff&rev=66764&r1=66763&r2=66764
==============================================================================
--- branches/1.2/channels/chan_sip.c (original)
+++ branches/1.2/channels/chan_sip.c Thu May 31 11:12:39 2007
@@ -3043,7 +3043,7 @@
 		return &null_frame;
 	if (p->owner) {
 		/* We already hold the channel lock */
-		if (f->frametype == AST_FRAME_VOICE) {
+		if (f && f->frametype == AST_FRAME_VOICE) {
 			if (f->subclass != p->owner->nativeformats) {
 				if (!(f->subclass & p->jointcapability)) {
 					ast_log(LOG_DEBUG, "Bogus frame of format '%s' received from '%s'!\n",