Opened 14 years ago
Closed 14 years ago
#2081 closed defect (fixed)
Some MOV chapters parsing broken since 0d96ec1
| Reported by: | Cigaes | Owned by: | Clément Bœsch |
|---|---|---|---|
| Priority: | important | Component: | avformat |
| Version: | unspecified | Keywords: | mov chapters regression |
| Cc: | Blocked By: | ||
| Blocking: | Reproduced by developer: | no | |
| Analyzed by developer: | yes |
Description
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).
Attachments (1)
Change History (3)
by , 14 years ago
| Attachment: | chapters.mov added |
|---|
comment:1 by , 14 years ago
| Owner: | set to |
|---|---|
| Status: | new → open |
comment:2 by , 14 years ago
| Resolution: | → fixed |
|---|---|
| Status: | open → closed |
Note:
See TracTickets
for help on using tickets.



Fixed by Hendrik in 765158dd.
Some cleanups are left to do (working on), but ticket can be closed. Thanks for the report.