[FFmpeg-devel] [PATCH 2/3] avformat/mov: Populate packet duration using stts atom instead of guessing

James Almer jamrial at gmail.com
Thu Nov 21 23:18:45 EET 2024


On 11/21/2024 6:17 PM, Michael Niedermayer wrote:
> On Thu, Nov 21, 2024 at 12:23:41AM -0300, James Almer wrote:
>> From: Darren Mo <fumoboy007 at me.com>
>>
>> Fixes tickets #7855 and #11312.
>>
>> Co-authored-by: James Almer <jamrial at gmail.com>
>> Signed-off-by: James Almer <jamrial at gmail.com>
>> ---
>> This is an alternative, better approach to the same issue.
>>
>>   libavformat/isom.h                            |   3 +
>>   libavformat/mov.c                             | 180 ++++++++++++++++--
>>   tests/ref/fate/copy-trac236                   |   4 +-
>>   tests/ref/fate/filter-fps                     |   1 +
>>   tests/ref/fate/filter-fps-cfr                 |   1 +
>>   tests/ref/fate/filter-meta-4560-rotate0       |   4 +-
>>   tests/ref/fate/gaplessenc-itunes-to-ipod-aac  |   2 +-
>>   tests/ref/fate/matroska-dovi-write-config8    |   4 +-
>>   .../fate/matroska-non-rotation-displaymatrix  |   6 +-
>>   tests/ref/fate/mov-aac-2048-priming           |   2 +-
>>   tests/ref/fate/mov-zombie                     |   4 +-
>>   11 files changed, 181 insertions(+), 30 deletions(-)
> 
> breaks fate
> 
> --- ./tests/ref/fate/mov-mp4-iamf-stereo	2024-11-17 02:14:20.475742935 +0100
> +++ tests/data/fate/mov-mp4-iamf-stereo	2024-11-21 22:14:37.421521040 +0100
> @@ -15,7 +15,7 @@
>   0,      27648,      27648,     4608,     1422, 0x2f9d47c5
>   0,      32256,      32256,     4608,     1768, 0x2a044b99
>   0,      36864,      36864,     4608,     1534, 0xb0b35a3f
> -0,      41472,      41472,     2628,      926, 0xc26a5eae
> +0,      41472,      41472,     4608,      926, 0xc26a5eae
>   [STREAM_GROUP]
>   index=0
>   id=0x1

There may be some use of uninitialized data somewhere. I'm looking at it.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature.asc
Type: application/pgp-signature
Size: 495 bytes
Desc: OpenPGP digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20241121/a185c207/attachment.sig>


More information about the ffmpeg-devel mailing list