diff options
| author | Andreas Rheinhardt <andreas.rheinhardt@gmail.com> | 2019-09-24 18:31:43 +0200 |
|---|---|---|
| committer | James Almer <jamrial@gmail.com> | 2019-09-25 20:53:01 -0300 |
| commit | e045be92cdf5a2851900e8e85b815c29ae6f100a (patch) | |
| tree | 3776e59517d046202b7dd77d9724f1139e1ae467 /libavformat/utils.c | |
| parent | c23797bc336c507188a502a5758b24a4595f14ec (diff) | |
| download | ffmpeg-e045be92cdf5a2851900e8e85b815c29ae6f100a.tar.gz | |
avformat/utils: Assert that stream_index is valid
There is currently an ordinary check for this (which would lead to a
memleak), but given that no demuxer should ever return a packet with an
invalid stream_index it is more appropriate for this to be an assert.
FATE passes with this change.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
Signed-off-by: James Almer <jamrial@gmail.com>
Diffstat (limited to 'libavformat/utils.c')
| -rw-r--r-- | libavformat/utils.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/libavformat/utils.c b/libavformat/utils.c index 3168931587..4d628119b6 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@ -884,10 +884,8 @@ int ff_read_packet(AVFormatContext *s, AVPacket *pkt) continue; } - if (pkt->stream_index >= (unsigned)s->nb_streams) { - av_log(s, AV_LOG_ERROR, "Invalid stream index %d\n", pkt->stream_index); - continue; - } + av_assert0(pkt->stream_index < (unsigned)s->nb_streams && + "Invalid stream index.\n"); st = s->streams[pkt->stream_index]; |