[FFmpeg-devel] [PATCH] lavu/opt: Fix return of uninitialised value

Marvin Scholz epirat07 at gmail.com
Wed Aug 28 04:08:39 EEST 2024


In one of the failure paths of av_opt_get_array, the ret variable
was accidentally declared again, shadowing the outer one and once when
jumping to the fail label would return the still uninitialised outer
one.

To fix this simply remove the local declaration of ret that shadowed
the outer one.

Introduced in d89930f8666c563a96f1dc1835b450522b4cd6da

Fixes: CID1618663 Uninitialized scalar variable
---
 libavutil/opt.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/libavutil/opt.c b/libavutil/opt.c
index d515e20e97..3788a9df75 100644
--- a/libavutil/opt.c
+++ b/libavutil/opt.c
@@ -2213,7 +2213,6 @@ int av_opt_get_array(void *obj, const char *name, int search_flags,
             double     num = 1.0;
             int        den = 1;
             int64_t intnum = 1;
-            int ret;
 
             ret = read_number(o, src, &num, &den, &intnum);
             if (ret < 0)

base-commit: d89930f8666c563a96f1dc1835b450522b4cd6da
-- 
2.39.3 (Apple Git-146)


More information about the ffmpeg-devel mailing list