[asterisk-bugs] [Asterisk 0019042]: [patch] [regression] H264 video broken due to MARK bit set for each RTP outgoing packet

[Asterisk Bug Tracker]

The following issue has been CLOSED 
====================================================================== 
https://issues.asterisk.org/view.php?id=19042 
====================================================================== 
Reported By:                festr
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   19042
Category:                   Core/RTP
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     closed
Asterisk Version:           SVN 
JIRA:                       SWP-3291 
Regression:                 Yes 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 311921 
Request Review:              
Resolution:                 open
Fixed in Version:           
====================================================================== 
Date Submitted:             2011-03-30 06:37 CDT
Last Modified:              2011-04-18 11:32 CDT
====================================================================== 
Summary:                    [patch] [regression] H264 video broken due to MARK
bit set for each RTP outgoing packet
Description: 
H264 video is broken because asterisk set MARK bit for every RTP packet and
video stream using more RTP packets for one frame (which is ended with MARK
packet) is broken.

I'm attaching patch which will check for mark bit. This check is in
asterisk-1.8, it is missing in asterisk-trunk. 
====================================================================== 

---------------------------------------------------------------------- 
 (0133879) rmudgett (administrator) - 2011-04-18 11:32
 https://issues.asterisk.org/view.php?id=19042#c133879 
---------------------------------------------------------------------- 
Author: twilson
Date: Thu Apr 14 16:53:01 2011
New Revision: 313822

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=313822
Log:
Sets video mark bit on format field correctly

This fixes a regression in the media architecture change
where video frames did not have their video mark set
correctly.  dvossel wrote this. twilson kindly committed
this, mmichelson found the bug.

Modified:
    trunk/res/res_rtp_asterisk.c 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2011-04-18 11:32 rmudgett       Note Added: 0133879                          
2011-04-18 11:32 rmudgett       Status                   ready for review =>
closed
======================================================================