[svn-commits] may: branch 1.6.0 r1115 - /branches/1.6.0/channels/chan_ooh323.c

[SVN commits to the Digium repositories]

Author: may
Date: Mon Jun  7 15:12:54 2010
New Revision: 1115

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

Modified: branches/1.6.0/channels/chan_ooh323.c
URL: http://svnview.digium.com/svn/asterisk-addons/branches/1.6.0/channels/chan_ooh323.c?view=diff&rev=1115&r1=1114&r2=1115
==============================================================================
--- branches/1.6.0/channels/chan_ooh323.c (original)
+++ branches/1.6.0/channels/chan_ooh323.c Mon Jun  7 15:12:54 2010
@@ -956,6 +956,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)