[FFmpeg-devel] [PATCH] MPEG sequeunce display extension parsing off by 2 error

[Michael Niedermayer]

On Mon, Jun 28, 2010 at 01:09:02PM -0400, Daniel Kristjansson wrote:
> 
> This isn't a very important patch since these are trailing bits,
> but if we're going to have a skip_bits call it should be skipping
> the correct number of bits.
> 
> -- Daniel 

>  mpeg12.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 0ad9e2599cd91781e496eeade79616dfd128a67d  mpeg-seq-disp-ext-marker-bits.patch
> Index: libavcodec/mpeg12.c
> ===================================================================
> --- libavcodec/mpeg12.c	(revision 23854)
> +++ libavcodec/mpeg12.c	(working copy)
> @@ -1440,7 +1440,7 @@
>      w= get_bits(&s->gb, 14);
>      skip_bits(&s->gb, 1); //marker
>      h= get_bits(&s->gb, 14);
> -    skip_bits(&s->gb, 1); //marker
> +    skip_bits(&s->gb, 3); //marker

my copy of the spec doesnt contain 3 marker bits
could you quote the relevant part of the spec please (including
which exact version that is)?


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No human being will ever know the Truth, for even if they happen to say it
by chance, they would not even known they had done so. -- Xenophanes
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100629/47d3e346/attachment.pgp>