On 01/09/2020 16:36, James Almer wrote:
> mfra_size in this function was an int32_t, so storing the output of
> avio_rb32() could end up with a negative value.
> It'll no longer be the case now that you made mfra_size in MOVContext an
> uint32_t.

How was that ever valid in the first place, considering avio_rb32 returns an
unsigned 32-bit integer?

- Derek
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to