On 8/7/2024 11:09 AM, Kacper Michajłow wrote:
Fixes: use-of-uninitialized-valueFound by OSS-Fuzz. --- libavformat/mov.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 1052691936..f2d8aee766 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -7096,7 +7096,7 @@ static int mov_read_free(MOVContext *c, AVIOContext *pb, MOVAtom atom) if (atom.size < 8) return 0;- ret = avio_read(pb, content, FFMIN(sizeof(content), atom.size));+ ret = ffio_read_size(pb, content, FFMIN(sizeof(content), atom.size)); if (ret < 0) return ret;
Unrelated (somewhat) to this patch, but why does ffio_read_size() replace EOF with INVALIDDATA? Is it a good idea to mask the former?
_______________________________________________ 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".
