[asterisk-addons-commits] may: branch 1.6.2 r1117 - /branches/1.6.2/channels/chan_ooh323.c

[SVN commits to the Asterisk addons project]

Author: may
Date: Mon Jun  7 15:15:09 2010
New Revision: 1117

URL: http://svnview.digium.com/svn/asterisk-addons?view=rev&rev=1117
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

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



Modified:
    branches/1.6.2/channels/chan_ooh323.c

Modified: branches/1.6.2/channels/chan_ooh323.c
URL: http://svnview.digium.com/svn/asterisk-addons/branches/1.6.2/channels/chan_ooh323.c?view=diff&rev=1117&r1=1116&r2=1117
==============================================================================
--- branches/1.6.2/channels/chan_ooh323.c (original)
+++ branches/1.6.2/channels/chan_ooh323.c Mon Jun  7 15:15:09 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)