[FFmpeg-devel] [PATCH v2] avformat/mov: Fix unaligned read of uint32_t and endian-dependance in mov_read_default

[Zhao Zhili]

Please help review the patch, thanks!

> On Jul 5, 2020, at 12:51 AM, Zhao Zhili <quinkblack at foxmail.com> wrote:
> 
> ---
> libavformat/mov.c | 11 +++++------
> 1 file changed, 5 insertions(+), 6 deletions(-)
> 
> diff --git a/libavformat/mov.c b/libavformat/mov.c
> index df5bebdff1..da438e4e2c 100644
> --- a/libavformat/mov.c
> +++ b/libavformat/mov.c
> @@ -6945,13 +6945,12 @@ static int mov_read_default(MOVContext *c, AVIOContext *pb, MOVAtom atom)
>                   a.type == MKTAG('h','o','o','v')) &&
>                 a.size >= 8 &&
>                 c->fc->strict_std_compliance < FF_COMPLIANCE_STRICT) {
> -                uint8_t buf[8];
> -                uint32_t *type = (uint32_t *)buf + 1;
> -                if (avio_read(pb, buf, 8) != 8)
> -                    return AVERROR_INVALIDDATA;
> +                uint32_t type;
> +                avio_skip(pb, 4);
> +                type = avio_rl32(pb);
>                 avio_seek(pb, -8, SEEK_CUR);
> -                if (*type == MKTAG('m','v','h','d') ||
> -                    *type == MKTAG('c','m','o','v')) {
> +                if (type == MKTAG('m','v','h','d') ||
> +                    type == MKTAG('c','m','o','v')) {
>                     av_log(c->fc, AV_LOG_ERROR, "Detected moov in a free or hoov atom.\n");
>                     a.type = MKTAG('m','o','o','v');
>                 }
> -- 
> 2.25.1
> 
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".