[FFmpeg-cvslog] fftools/ffmpeg: Don't presume frame_queue to have been allocated
Andreas Rheinhardt
git at videolan.org
Thu Mar 3 04:59:50 EET 2022
ffmpeg | branch: master | Andreas Rheinhardt <andreas.rheinhardt at outlook.com> | Wed Feb 23 20:11:51 2022 +0100| [6559858de9a3c18bd5cf4892cb7b5b90180e7d97] | committer: Andreas Rheinhardt
fftools/ffmpeg: Don't presume frame_queue to have been allocated
Fixes segfaults upon allocation failure.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=6559858de9a3c18bd5cf4892cb7b5b90180e7d97
---
fftools/ffmpeg.c | 10 ++++++----
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/fftools/ffmpeg.c b/fftools/ffmpeg.c
index d5cd010a9c..9a3fdc636d 100644
--- a/fftools/ffmpeg.c
+++ b/fftools/ffmpeg.c
@@ -527,11 +527,13 @@ static void ffmpeg_cleanup(int ret)
for (j = 0; j < fg->nb_inputs; j++) {
InputFilter *ifilter = fg->inputs[j];
struct InputStream *ist = ifilter->ist;
- AVFrame *frame;
- while (av_fifo_read(ifilter->frame_queue, &frame, 1) >= 0)
- av_frame_free(&frame);
- av_fifo_freep2(&ifilter->frame_queue);
+ if (ifilter->frame_queue) {
+ AVFrame *frame;
+ while (av_fifo_read(ifilter->frame_queue, &frame, 1) >= 0)
+ av_frame_free(&frame);
+ av_fifo_freep2(&ifilter->frame_queue);
+ }
av_freep(&ifilter->displaymatrix);
if (ist->sub2video.sub_queue) {
AVSubtitle sub;
More information about the ffmpeg-cvslog
mailing list