diff options
| author | Vittorio Giovara <vittorio.giovara@gmail.com> | 2021-04-06 18:15:49 +0200 |
|---|---|---|
| committer | Vittorio Giovara <vittorio.giovara@gmail.com> | 2021-04-15 16:21:58 +0200 |
| commit | 918273aa3eb721323ec2de37a39ad8ce9f4dcfb7 (patch) | |
| tree | 8d65711f7a46f5b0b401a6769027d265bdf00197 | |
| parent | 4e64c8fa29bb2777098c29887d8a0e7b8b0d514d (diff) | |
| download | ffmpeg-918273aa3eb721323ec2de37a39ad8ce9f4dcfb7.tar.gz | |
mov: Prioritize aspect ratio values found in pasp atom
From the ISO/IEC specification for MP4:
The pixel aspect ratio and clean aperture of the video may be specified
using the ‘pasp’ and ‘clap’ sample entry boxes, respectively. These are
both optional; if present, they over-ride the declarations (if any) in
structures specific to the video codec, which structures should be
examined if these boxes are absent. For maximum compatibility, these
boxes should follow, not precede, any boxes defined in or required by
derived specifications.
Fixes trac/#7277.
| -rw-r--r-- | libavformat/mov.c | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/libavformat/mov.c b/libavformat/mov.c index 43c55ef4a4..9ca1ac89a8 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -941,6 +941,7 @@ static int mov_read_wfex(MOVContext *c, AVIOContext *pb, MOVAtom atom) return ret; } +/* This atom overrides any previously set aspect ratio */ static int mov_read_pasp(MOVContext *c, AVIOContext *pb, MOVAtom atom) { const int num = avio_rb32(pb); @@ -951,13 +952,7 @@ static int mov_read_pasp(MOVContext *c, AVIOContext *pb, MOVAtom atom) return 0; st = c->fc->streams[c->fc->nb_streams-1]; - if ((st->sample_aspect_ratio.den != 1 || st->sample_aspect_ratio.num) && // default - (den != st->sample_aspect_ratio.den || num != st->sample_aspect_ratio.num)) { - av_log(c->fc, AV_LOG_WARNING, - "sample aspect ratio already set to %d:%d, ignoring 'pasp' atom (%d:%d)\n", - st->sample_aspect_ratio.num, st->sample_aspect_ratio.den, - num, den); - } else if (den != 0) { + if (den != 0) { av_reduce(&st->sample_aspect_ratio.num, &st->sample_aspect_ratio.den, num, den, 32767); } |