[FFmpeg-devel] [PATCH] avcodec/libuavs3d: fix access uninitialized variable when draining

[mypopy at gmail.com]

On Thu, May 12, 2022 at 5:55 PM Zhao Zhili <quinkblack at foxmail.com> wrote:
>
> buf_ptr is uninitialized and accessed when function return.
> ---
>  libavcodec/libuavs3d.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/libavcodec/libuavs3d.c b/libavcodec/libuavs3d.c
> index 6966e00b62..23de4c8cd5 100644
> --- a/libavcodec/libuavs3d.c
> +++ b/libavcodec/libuavs3d.c
> @@ -149,7 +149,7 @@ static int libuavs3d_decode_frame(AVCodecContext *avctx, AVFrame *frm,
>      const uint8_t *buf = avpkt->data;
>      int buf_size = avpkt->size;
>      const uint8_t *buf_end;
> -    const uint8_t *buf_ptr;
> +    const uint8_t *buf_ptr = buf;
>      int left_bytes;
>      int ret, finish = 0;
>
> @@ -170,7 +170,6 @@ static int libuavs3d_decode_frame(AVCodecContext *avctx, AVFrame *frm,
>      } else {
>          uavs3d_io_frm_t *frm_dec = &h->dec_frame;
>
> -        buf_ptr = buf;
>          buf_end = buf + buf_size;
>          frm_dec->pkt_pos  = avpkt->pos;
>          frm_dec->pkt_size = avpkt->size;
> --
> 2.35.3
Sounds Good To Me