[FFmpeg-cvslog] avdevice/decklink_dec: fix leak on error

Marvin Scholz git at videolan.org
Sun Sep 29 11:30:01 EEST 2024


ffmpeg | branch: master | Marvin Scholz <epirat07 at gmail.com> | Thu Sep 12 04:52:43 2024 +0200| [7554346cb5cf6e8863da12f9d56e300b08d6cc23] | committer: Marton Balint

avdevice/decklink_dec: fix leak on error

In the early return when both draw_bars and signal_loss_action
options are used, the context allocated previously was not
properly freed.

Introduced in 9bcb86b0fa58f1a5a2a8613065349e26abadb329

Fixes CID1619296

Signed-off-by: Marton Balint <cus at passwd.hu>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=7554346cb5cf6e8863da12f9d56e300b08d6cc23
---

 libavdevice/decklink_dec.cpp | 1 +
 1 file changed, 1 insertion(+)

diff --git a/libavdevice/decklink_dec.cpp b/libavdevice/decklink_dec.cpp
index 54d698bc12..9a817daf18 100644
--- a/libavdevice/decklink_dec.cpp
+++ b/libavdevice/decklink_dec.cpp
@@ -1098,6 +1098,7 @@ av_cold int ff_decklink_read_header(AVFormatContext *avctx)
     }
     if (!ctx->draw_bars && ctx->signal_loss_action != SIGNAL_LOSS_NONE) {
         av_log(avctx, AV_LOG_ERROR, "options draw_bars and signal_loss_action are mutually exclusive\n");
+        av_freep(&ctx);
         return AVERROR(EINVAL);
     }
     ctx->audio_depth = cctx->audio_depth;



More information about the ffmpeg-cvslog mailing list