[FFmpeg-devel] [PATCH] mov: add support for little-endian utf16 chapter names

Janne Grunau janne-ffmpeg
Thu Jan 27 18:20:52 CET 2011


On Thu, Jan 27, 2011 at 05:11:09PM +0000, M?ns Rullg?rd wrote:
> Janne Grunau <janne-ffmpeg at jannau.net> writes:
> 
> > From: John Stebbins <stebbins at jetheaddev.com>
> >
> > refreshed patch against master
> >
> > Janne
> >
> > -- 8< --
> > fixes a regressin in 225b6d7
> > "mov: simplify mov_read_chapters() by using avio_get_str16be"
> >
> > Signed-off-by: Janne Grunau <janne-ffmpeg at jannau.net>
> > ---
> >  libavformat/mov.c |    2 ++
> >  1 files changed, 2 insertions(+), 0 deletions(-)
> >
> > diff --git a/libavformat/mov.c b/libavformat/mov.c
> > index ea9423b..7ad920f 100644
> > --- a/libavformat/mov.c
> > +++ b/libavformat/mov.c
> > @@ -2326,6 +2326,8 @@ static void mov_read_chapters(AVFormatContext *s)
> >          ch = get_be16(sc->pb);
> >          if (ch == 0xfeff)
> >              avio_get_str16be(sc->pb, len, title, title_len);
> > +        else if (ch == 0xfffe)
> > +            avio_get_str16le(sc->pb, len, title, title_len);
> >          else {
> >              AV_WB16(title, ch);
> >              get_strz(sc->pb, title + 2, len - 1);
> > -- 
> > 1.7.4.rc2
> 
> Looks good.

queued

Janne



More information about the ffmpeg-devel mailing list