[asterisk-addons-commits] may: branch 1.6.1 r1116 - /branches/1.6.1/channels/chan_ooh323.c

SVN commits to the Asterisk addons project asterisk-addons-commits at lists.digium.com
Mon Jun 7 15:14:14 CDT 2010


Author: may
Date: Mon Jun  7 15:14:10 2010
New Revision: 1116

URL: http://svnview.digium.com/svn/asterisk-addons?view=rev&rev=1116
Log:
don't read rtp data from channel without private structure

In some cases asterisk channel can exist but on protocol stack side
channel can
be closed and we must return null frame in reading data functions
instead of reading data from destroyed channel

(issue #17227)
Reported by: jin
Patches:
      bug17227.patch uploaded by may213 (license 454)
Tested by: jin


Modified:
    branches/1.6.1/channels/chan_ooh323.c

Modified: branches/1.6.1/channels/chan_ooh323.c
URL: http://svnview.digium.com/svn/asterisk-addons/branches/1.6.1/channels/chan_ooh323.c?view=diff&rev=1116&r1=1115&r2=1116
==============================================================================
--- branches/1.6.1/channels/chan_ooh323.c (original)
+++ branches/1.6.1/channels/chan_ooh323.c Mon Jun  7 15:14:10 2010
@@ -886,6 +886,9 @@
 	struct ast_frame *fr;
 	static struct ast_frame null_frame = { AST_FRAME_NULL, };
 	struct ooh323_pvt *p = ast->tech_pvt;
+
+	if (!p)
+		return &null_frame;
 
 	ast_mutex_lock(&p->lock);
 	if (p->rtp)




More information about the asterisk-addons-commits mailing list