RepoMirrors/ffmpeg
1
0
Fork 0
mirror of https://git.ffmpeg.org/ffmpeg.git synced 2025-01-10 09:09:50 +00:00
Code Issues Projects Releases Wiki Activity

libavformat/oggdec.c: Check return value from avio_read()

Browse Source 
If the buffer doesn't contain enough bytes when reading a stream,
fail rather than continuing on with unitialized data. Caught by
Chromium fuzzers (crbug.com/1054229).

Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
(cherry picked from commit b7c67b1ae3)
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
This commit is contained in:
John Rummell 2020-03-30 14:56:11 -07:00 committed by Michael Niedermayer
parent 52dbafe7b6
commit 33bdb19d23
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
libavformat/oggdec.c
View File

@ -216,7 +216,8 @@ static int ogg_replace_stream(AVFormatContext *s, uint32_t serial, int nsegs)
uint8_t magic[8]; uint8_t magic[8];
int64_t pos = avio_tell(s->pb); int64_t pos = avio_tell(s->pb);
avio_skip(s->pb, nsegs); avio_skip(s->pb, nsegs);
avio_read(s->pb, magic, sizeof(magic)); if (avio_read(s->pb, magic, sizeof(magic)) != sizeof(magic))
return AVERROR_INVALIDDATA;
avio_seek(s->pb, pos, SEEK_SET); avio_seek(s->pb, pos, SEEK_SET);
codec = ogg_find_codec(magic, sizeof(magic)); codec = ogg_find_codec(magic, sizeof(magic));
if (!codec) { if (!codec) {