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) _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".