mirror of
https://github.com/FFmpeg/FFmpeg.git
synced 2025-03-17 20:17:55 +02:00
avformat/4xm: Check that a video stream was created before returning packets for it
Fixes: assertion failure Fixes: 23434/clusterfuzz-testcase-minimized-ffmpeg_DEMUXER_fuzzer-5227750851084288.fuzz Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer <michael@niedermayer.cc> (cherry picked from commit c517c3f4741b6897ea952d1fba199c93c5217cfe) Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
This commit is contained in:
parent
bd103bd3fe
commit
2e3f6afda5
@ -218,6 +218,7 @@ static int fourxm_read_header(AVFormatContext *s)
|
|||||||
fourxm->track_count = 0;
|
fourxm->track_count = 0;
|
||||||
fourxm->tracks = NULL;
|
fourxm->tracks = NULL;
|
||||||
fourxm->fps = (AVRational){1,1};
|
fourxm->fps = (AVRational){1,1};
|
||||||
|
fourxm->video_stream_index = -1;
|
||||||
|
|
||||||
/* skip the first 3 32-bit numbers */
|
/* skip the first 3 32-bit numbers */
|
||||||
avio_skip(pb, 12);
|
avio_skip(pb, 12);
|
||||||
@ -323,6 +324,8 @@ static int fourxm_read_packet(AVFormatContext *s,
|
|||||||
case cfr2_TAG:
|
case cfr2_TAG:
|
||||||
/* allocate 8 more bytes than 'size' to account for fourcc
|
/* allocate 8 more bytes than 'size' to account for fourcc
|
||||||
* and size */
|
* and size */
|
||||||
|
if (fourxm->video_stream_index < 0)
|
||||||
|
return AVERROR_INVALIDDATA;
|
||||||
if (size + 8 < size || av_new_packet(pkt, size + 8))
|
if (size + 8 < size || av_new_packet(pkt, size + 8))
|
||||||
return AVERROR(EIO);
|
return AVERROR(EIO);
|
||||||
pkt->stream_index = fourxm->video_stream_index;
|
pkt->stream_index = fourxm->video_stream_index;
|
||||||
|
Loading…
x
Reference in New Issue
Block a user