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

List:       asterisk-bugs
Subject:    [asterisk-bugs] [Asterisk 0011429]: Sent RTP video packets have a
From:       noreply () carolina ! digium ! com
Date:       2007-11-30 23:30:53
Message-ID: 70e37f956c525f57bbd5d22f08bb5c20 () bugs ! digium ! com
[Download RAW message or body]


A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=11429 
====================================================================== 
Reported By:                sperreault
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   11429
Category:                   Core/RTP
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     new
Asterisk Version:            SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 90351 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             11-30-2007 14:53 CST
Last Modified:              11-30-2007 17:30 CST
====================================================================== 
Summary:                    Sent RTP video packets have a timestamp based on a
8000 Hz clock instead of 90000 Hz when mark bit is on
Description: 
When you ast_write() a video packet, you are supposed to put the mark bit
into the LSB of the subclass member:

f->subclass = AST_FORMAT_H263 | 1;

This makes this conditional in ast_rtp_raw_write() true while it should be
false:

if (f->subclass & AST_FORMAT_AUDIO_MASK) {

The solution is simple: invert the video and audio code paths. First check
for video, and then if video is not present check for audio.
====================================================================== 

---------------------------------------------------------------------- 
 sperreault - 11-30-07 17:30  
---------------------------------------------------------------------- 
Here's a silly answer: I don't know! I just wanted a quick fix with no
chances of breaking anything. I think it should be investigated whether we
can use frametype or not, but should be done separately from fixing this
bug. 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
11-30-07 17:30  sperreault     Note Added: 0074621                          
======================================================================


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

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

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