[asterisk-dev] [svn-commits] twilson: trunk r313822 - /trunk/res/res_rtp_asterisk.c

[Martin Vít]

Hi,

I've found this bug with the same fix laying on issues.asterisk.org - 
https://issues.asterisk.org/view.php?id=19042

So you can close this and do reletions to it.

Festr

On 14.4.2011 23:53, SVN commits to the Digium repositories wrote:
> 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
>
> Modified: trunk/res/res_rtp_asterisk.c
> URL: http://svnview.digium.com/svn/asterisk/trunk/res/res_rtp_asterisk.c?view=diff&rev=313822&r1=313821&r2=313822
> ==============================================================================
> --- trunk/res/res_rtp_asterisk.c (original)
> +++ trunk/res/res_rtp_asterisk.c Thu Apr 14 16:53:01 2011
> @@ -2311,7 +2311,9 @@
>   		rtp->f.delivery.tv_sec = 0;
>   		rtp->f.delivery.tv_usec = 0;
>   		/* Pass the RTP marker bit as bit */
> -		ast_format_set_video_mark(&rtp->f.subclass.format);
> +		if (mark) {
> +			ast_format_set_video_mark(&rtp->f.subclass.format);
> +		}
>   	} else {
>   		/* TEXT -- samples is # of samples vs. 1000 */
>   		if (!rtp->lastitexttimestamp)
>
>
> --
> _____________________________________________________________________
> -- Bandwidth and Colocation Provided by http://www.api-digital.com --
>
> svn-commits mailing list
> To UNSUBSCRIBE or update options visit:
>     http://lists.digium.com/mailman/listinfo/svn-commits
>


-- 
Mgr. Martin Vít
hlasové služby

LAM plus s.r.o.
Mezi vodami 19
143 00 Praha - Modřany
http://www.lam.cz
+420 242 413 000
+420 605 267 610