[FFmpeg-devel] [PATCH v2] mov: Properly abide by the track's media duration

Derek Buitenhuis derek.buitenhuis at gmail.com
Wed Apr 25 18:07:31 EEST 2018


On 4/23/2018 4:46 PM, Derek Buitenhuis wrote:
> The track's media duration from the mdhd atom takes precedence
> over both the stts and elst atom for calculating and setting
> the track's total duraion.
> 
> Technically, we shouldn't be using the stts atom at all for
> calculating stream durations.
> 
> This fixes incorrect stream and final packet durations on files
> with edit lists that are longer than the media duration.
> 
> The FATE changes are expected, and output is more correct (the
> AAC frame is not 1028 samples).
> 
> Signed-off-by: Derek Buitenhuis <derek.buitenhuis at gmail.com>
> ---
>  libavformat/mov.c                   | 6 +++---
>  tests/ref/fate/adtstoasc_ticket3715 | 2 +-
>  2 files changed, 4 insertions(+), 4 deletions(-)

Ping.

If there are no further comments, I plan to push tomorrow evening.

- Derek


More information about the ffmpeg-devel mailing list