diff options
| author | Andreas Rheinhardt <andreas.rheinhardt@outlook.com> | 2021-05-15 08:51:11 +0200 |
|---|---|---|
| committer | Andreas Rheinhardt <andreas.rheinhardt@outlook.com> | 2021-06-08 12:52:50 +0200 |
| commit | d29541c90aaa2e16d25411ba73f96576da99e0e1 (patch) | |
| tree | a6ed594bfd9d271735569908234546f0125ba061 | |
| parent | 3a5412b39eeb2f7af66acc985b55ccdb8bb9a953 (diff) | |
| download | ffmpeg-d29541c90aaa2e16d25411ba73f96576da99e0e1.tar.gz | |
fftools/ffprobe: Remove redundant check before avcodec_free_context()
Besides being unnecessary it is also safer: If the error for an
unrecognized option were triggered (which seems to be impossible right
now), it might be that the stream whose codecpar is accessed is NULL.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
| -rw-r--r-- | fftools/ffprobe.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fftools/ffprobe.c b/fftools/ffprobe.c index 88e82a4195..2d452c212e 100644 --- a/fftools/ffprobe.c +++ b/fftools/ffprobe.c @@ -3078,8 +3078,7 @@ static void close_input_file(InputFile *ifile) /* close decoder for each stream */ for (i = 0; i < ifile->nb_streams; i++) - if (ifile->streams[i].st->codecpar->codec_id != AV_CODEC_ID_NONE) - avcodec_free_context(&ifile->streams[i].dec_ctx); + avcodec_free_context(&ifile->streams[i].dec_ctx); av_freep(&ifile->streams); ifile->nb_streams = 0; |