[FFmpeg-devel] Possible bug in dvbsub.c
Wed Feb 9 18:29:15 CET 2011
On Tue, Feb 8, 2011 at 4:52 PM, Janne Grunau <janne-ffmpeg at jannau.net>wrote:
> On Tue, Feb 08, 2011 at 07:14:58PM +0000, M?ns Rullg?rd wrote:
> > Ronen Mizrahi <ronen at tversity.com> writes:
> > > Hello,
> > >
> > > Looks like line 226 in dvbsub.c is neglecting to put the object version
> > > the correct bits:
> > > *q++ = s->object_version | (page_state << 2) | 3;
> > >
> > > It should be:
> > >
> > > *q++ = (s->object_version << 4) | (page_state << 2) | 3;
> > That certainly _looks_ more reasonable. Care to prepare a proper patch?
> yes, it's correct
> > > I am no dvbsub expert so I was hoping someone on the list can confirm.
> In my
> > > case the encoding did not play correctly in vlc before this patch but
> > > after it, so I suspect it is indeed required.
> > Which spec covers this? Shouldn't be hard to verify the fix against
> ETSI EN 300 743 available for download from http://www.dvb.org
> ffmpeg-devel mailing list
> ffmpeg-devel at mplayerhq.hu
Please see the patch attached.
I did verify it against the spec and against vlc source code at:
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 413 bytes
Desc: not available
More information about the ffmpeg-devel