ffmpeg | branch: master | Marvin Scholz <epira...@gmail.com> | Sat Aug 31 22:45:31 2024 +0200| [b6a0eab528695c39a0c52889db0c1ce5dd6d99f3] | committer: James Almer
avformat/iamf_parse: Fix return of uninitialized value The ret value here is not yet intialized so the return would return uninitialized data. What was probably meant to be checked here was the return value of ffio_read_size, which can return an error. Introduced in 38bcb3ba7b3424abd772c72f8bdf445d75285e88 Fixes: CID1618758 Signed-off-by: James Almer <jamr...@gmail.com> > http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=b6a0eab528695c39a0c52889db0c1ce5dd6d99f3 --- libavformat/iamf_parse.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavformat/iamf_parse.c b/libavformat/iamf_parse.c index f13e76b147..23170f504e 100644 --- a/libavformat/iamf_parse.c +++ b/libavformat/iamf_parse.c @@ -97,9 +97,10 @@ static int aac_decoder_config(IAMFCodecConfig *codec_config, if (!codec_config->extradata) return AVERROR(ENOMEM); - codec_config->extradata_size = ffio_read_size(pb, codec_config->extradata, left); + ret = ffio_read_size(pb, codec_config->extradata, left); if (ret < 0) return ret; + codec_config->extradata_size = left; memset(codec_config->extradata + codec_config->extradata_size, 0, AV_INPUT_BUFFER_PADDING_SIZE); _______________________________________________ ffmpeg-cvslog mailing list ffmpeg-cvslog@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-cvslog To unsubscribe, visit link above, or email ffmpeg-cvslog-requ...@ffmpeg.org with subject "unsubscribe".
