The branch, master has been updated
via 32cefd3789a26b263c87cb06a6aec4a4b2d171bb (commit)
from 0ae8df5f2ceea82337a2456ef16f930faf160189 (commit)
- Log -----------------------------------------------------------------
commit 32cefd3789a26b263c87cb06a6aec4a4b2d171bb
Author: James Zern <[email protected]>
AuthorDate: Wed Nov 19 10:38:58 2025 -0800
Commit: James Zern <[email protected]>
CommitDate: Wed Nov 19 12:43:12 2025 -0800
avcodec/libaomenc: Fix use of uninitialized value
codecctl_intp() is used to populate values via aom_codec_control(), not
set them. This change moves the output of the return pointer's value
after the aom_codec_control() to avoid a memory sanitizer warning (use
of uninitialized value).
Signed-off-by: James Zern <[email protected]>
diff --git a/libavcodec/libaomenc.c b/libavcodec/libaomenc.c
index 9fb97e74ff..1e5db4ad8c 100644
--- a/libavcodec/libaomenc.c
+++ b/libavcodec/libaomenc.c
@@ -392,17 +392,17 @@ static av_cold int codecctl_intp(AVCodecContext *avctx,
int width = -30;
int res;
- snprintf(buf, sizeof(buf), "%s:", ctlidstr[id]);
- av_log(avctx, AV_LOG_DEBUG, " %*s%d\n", width, buf, *ptr);
-
res = aom_codec_control(&ctx->encoder, id, ptr);
if (res != AOM_CODEC_OK) {
- snprintf(buf, sizeof(buf), "Failed to set %s codec control",
+ snprintf(buf, sizeof(buf), "Failed to get %s codec control",
ctlidstr[id]);
log_encoder_error(avctx, buf);
return AVERROR(EINVAL);
}
+ snprintf(buf, sizeof(buf), "%s:", ctlidstr[id]);
+ av_log(avctx, AV_LOG_DEBUG, " %*s%d\n", width, buf, *ptr);
+
return 0;
}
#endif
-----------------------------------------------------------------------
Summary of changes:
libavcodec/libaomenc.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
hooks/post-receive
--
_______________________________________________
ffmpeg-cvslog mailing list -- [email protected]
To unsubscribe send an email to [email protected]
