Mans Rullgard <[EMAIL PROTECTED]> added the comment:
Michael Niedermayer <[EMAIL PROTECTED]> writes:
> Michael Niedermayer <[EMAIL PROTECTED]> added the comment:
>
> Suggested patch attached.
> I will commit in 48h if no objections.
>
> Index: mpeg.c
> ===================================================================
> --- mpeg.c (revision 12265)
> +++ mpeg.c (working copy)
> @@ -334,6 +334,10 @@
> header_len -= 5;
> }
> }
> + if (flags & 0x01 && header_len == 0){
> + flags &= ~0x01;
> + av_log(s, AV_LOG_WARNING, "PES Extension flag set but no bytes
> left\n");
> + }
This could just as well cover all PES header parts signalled by flags,
even though we don't care about them.
> if (flags & 0x01) { /* PES extension */
> pes_ext = get_byte(s->pb);
> header_len--;
This bit of code could do with a little more error checking...
______________________________________________________
FFmpeg issue tracker <[EMAIL PROTECTED]>
<https://roundup.mplayerhq.hu/roundup/ffmpeg/issue198>
______________________________________________________
