Hi, width and height use u(16) descriptor in prores specification, and spec does not limit scope, I guess 1~65535 are allowable. So I think I'll change it to check if they are equal to 0. What do you propose?
-----邮件原件----- 发件人: ffmpeg-devel <[email protected]> 代表 Tomas H?rdin 发送时间: 2023年7月25日 19:48 收件人: FFmpeg development discussions and patches <[email protected]> 主题: Re: [FFmpeg-devel] [PATCH v2] add prores bitstream demuxer and muxer > +static int prores_check_frame_header(const uint8_t *buf, const int > data_size) > +{ > + int hdr_size, width, height; > + int version, alpha_info; > + > + hdr_size = AV_RB16(buf); > + if (hdr_size < FRAME_FIXED_HEADER_SIZE) > + return AVERROR_INVALIDDATA; > + > + version = buf[3]; > + if (version > 1) > + return AVERROR_INVALIDDATA; > + > + width = AV_RB16(buf + 8); > + height = AV_RB16(buf + 10); > + if (width < 16 || height < 16) As others pointed out, is there a maximum width or height also, or perhaps a maximum width*height? /Tomas _______________________________________________ ffmpeg-devel mailing list [email protected] https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email [email protected] with subject "unsubscribe". _______________________________________________ ffmpeg-devel mailing list [email protected] https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email [email protected] with subject "unsubscribe".
