[FFmpeg-devel] [PR] avformat/whip: Avoid uninitialized pointer read (PR #21804)

[FFmpeg-devel] [PR] avformat/whip: Avoid uninitialized pointer read (PR #21804)

[vigneshvg via ffmpeg-devel]

PR #21804 opened by vigneshvg
URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/21804
Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/21804.patch

Do not try to free the io context if it was never allocated.

Fixes coverity #1665383

Signed-off-by: Vignesh Venkatasubramanian <vigneshv@google.com>


>From 98f604e096714cfe22518a7a961541a58cc128bd Mon Sep 17 00:00:00 2001
From: Vignesh Venkat <vigneshv@google.com>
Date: Thu, 19 Feb 2026 10:58:00 -0800
Subject: [PATCH] avformat/whip: Avoid uninitialized pointer read

Do not try to free the io context if it was never allocated.

Fixes coverity #1665383

Signed-off-by: Vignesh Venkatasubramanian <vigneshv@google.com>
---
 libavformat/whip.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/libavformat/whip.c b/libavformat/whip.c
index 8aed0c31e5..fda5fed40c 100644
--- a/libavformat/whip.c
+++ b/libavformat/whip.c
@@ -892,8 +892,7 @@ static int parse_answer(AVFormatContext *s)
 
     if (!whip->sdp_answer || !strlen(whip->sdp_answer)) {
         av_log(whip, AV_LOG_ERROR, "No answer to parse\n");
-        ret = AVERROR(EINVAL);
-        goto end;
+        return AVERROR(EINVAL);
     }
 
     pb = avio_alloc_context(whip->sdp_answer, strlen(whip->sdp_answer), 0, NULL, NULL, NULL, NULL);
-- 
2.52.0

_______________________________________________
ffmpeg-devel mailing list -- ffmpeg-devel@ffmpeg.org
To unsubscribe send an email to ffmpeg-devel-leave@ffmpeg.org