[prev in list] [next in list] [prev in thread] [next in thread] 

List:       asterisk-addons-commits
Subject:    [asterisk-addons-commits] may: branch 1.6.1 r1116 -
From:       SVN commits to the Asterisk addons project
Date:       2010-06-07 20:14:14
Message-ID: E1OLiif-00021a-Mj () lists ! digium ! com
[Download RAW message or body]

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)


-- 
_____________________________________________________________________
-- Bandwidth and Colocation Provided by http://www.api-digital.com --

asterisk-addons-commits mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-addons-commits


[prev in list] [next in list] [prev in thread] [next in thread]