[FFmpeg-devel] [PATCH] avcodec/mediacodecenc: Fix access of uninitialized value
Matthieu Bouron
matthieu.bouron at gmail.com
Sun Oct 6 10:28:08 EEST 2024
On Thu, Oct 3, 2024 at 7:31 PM Zhao Zhili <quinkblack at foxmail.com> wrote:
>
> From: Zhao Zhili <zhilizhao at tencent.com>
>
> When crop is skipped, av_strlcatf will access `str` which isn't
> initialized properly.
> ---
> libavcodec/mediacodecenc.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavcodec/mediacodecenc.c b/libavcodec/mediacodecenc.c
> index 6ca3968a24..e76ea81236 100644
> --- a/libavcodec/mediacodecenc.c
> +++ b/libavcodec/mediacodecenc.c
> @@ -134,7 +134,7 @@ static int extract_extradata_support(AVCodecContext *avctx)
> static int mediacodec_init_bsf(AVCodecContext *avctx)
> {
> MediaCodecEncContext *s = avctx->priv_data;
> - char str[128];
> + char str[128] = {0};
> int ret;
> int crop_right = s->width - avctx->width;
> int crop_bottom = s->height - avctx->height;
> --
> 2.46.0
LGTM.
More information about the ffmpeg-devel
mailing list