[FFmpeg-trac] #2081(avformat:new): Some MOV chapters parsing broken since 0d96ec1
FFmpeg
trac at avcodec.org
Sun Dec 30 10:12:29 CET 2012
#2081: Some MOV chapters parsing broken since 0d96ec1
-------------------------------------+-------------------------------------
Reporter: Cigaes | Owner:
Type: defect | Status: new
Priority: important | Component: avformat
Version: | Keywords: mov
unspecified | chapters regression
Blocked By: | Blocking:
Reproduced by developer: 0 | Analyzed by developer: 1
-------------------------------------+-------------------------------------
With the attached sample:
{{{
./ffmpeg_g -i /tmp/chapters.mov
}}}
Expected (truncated) outout:
{{{
Input #0, mov,mp4,m4a,3gp,3g2,mj2, from '/tmp/chapters.mov':
Metadata:
major_brand : qt
minor_version : 537199360
compatible_brands: qt
creation_time : 2012-12-24 00:08:45
Duration: 00:21:21.96, start: 0.000000, bitrate: 3 kb/s
Chapter #0.0: start 0.000000, end 0.480000
Metadata:
title :
Stream #0:0(eng): Video: h264 (avc1 / 0x31637661), 640x480, 1694 kb/s,
25 fps, 25 tbr, 2500 tbn, 5k tbc
}}}
Actual output:
{{{
Input #0, mov,mp4,m4a,3gp,3g2,mj2, from '/tmp/chapters.mov':
Metadata:
major_brand : qt
minor_version : 537199360
compatible_brands: qt
creation_time : 2012-12-24 00:08:45
Duration: 00:21:21.96, start: 0.000000, bitrate: 3 kb/s
Stream #0:0(eng): Video: h264 (avc1 / 0x31637661), 640x480, 1694 kb/s,
25 fps, 25 tbr, 2500 tbn, 5k tbc
}}}
Notice the chapter just after "duration". The change comes from the
following commit:
{{{0d96ec1 mov: parse tref atom.}}}
Reverting it (or just replacing {{{mov_read_tref}}} with
{{{mov_read_default}}} in
{{{libavformat/mov.c}}}:{{{mov_default_parse_table}}} restores the correct
behaviour (but certainly breaks the feature it was adding).
--
Ticket URL: <https://ffmpeg.org/trac/ffmpeg/ticket/2081>
FFmpeg <http://ffmpeg.org>
FFmpeg issue tracker
More information about the FFmpeg-trac
mailing list