[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <8e4bde65-fe66-853b-8b87-f3b230a384df@quicinc.com>
Date: Wed, 17 May 2023 10:10:25 -0700
From: Carl Vanderlip <quic_carlv@...cinc.com>
To: Jeffrey Hugo <quic_jhugo@...cinc.com>, <trix@...hat.com>,
<ogabbay@...nel.org>, <nathan@...nel.org>,
<ndesaulniers@...gle.com>, <jacek.lawrynowicz@...ux.intel.com>,
<stanislaw.gruszka@...ux.intel.com>, <quic_pkanojiy@...cinc.com>
CC: <linux-arm-msm@...r.kernel.org>, <dri-devel@...ts.freedesktop.org>,
<linux-kernel@...r.kernel.org>, <llvm@...ts.linux.dev>
Subject: Re: [PATCH v2] accel/qaic: initialize ret variable to 0
On 5/17/2023 9:56 AM, Jeffrey Hugo wrote:
> From: Tom Rix <trix@...hat.com>
>
> clang static analysis reports
> drivers/accel/qaic/qaic_data.c:610:2: warning: Undefined or garbage
> value returned to caller [core.uninitialized.UndefReturn]
> return ret;
> ^~~~~~~~~~
>
>>>From a code analysis of the function, the ret variable is only set some
> of the time but is always returned. This suggests ret can return
> uninitialized garbage. However BO allocation will ensure ret is always
> set in reality.
>
> Initialize ret to 0 to silence the warning.
>
> Fixes: ff13be830333 ("accel/qaic: Add datapath")
> Signed-off-by: Tom Rix <trix@...hat.com>
> [jhugo: Reword commit text]
> Signed-off-by: Jeffrey Hugo <quic_jhugo@...cinc.com>
> ---
> drivers/accel/qaic/qaic_data.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/accel/qaic/qaic_data.c b/drivers/accel/qaic/qaic_data.c
> index 8ab26e64b231..e42c1f9ffff8 100644
> --- a/drivers/accel/qaic/qaic_data.c
> +++ b/drivers/accel/qaic/qaic_data.c
> @@ -591,7 +591,7 @@ static int qaic_gem_object_mmap(struct drm_gem_object *obj, struct vm_area_struc
> struct qaic_bo *bo = to_qaic_bo(obj);
> unsigned long offset = 0;
> struct scatterlist *sg;
> - int ret;
> + int ret = 0;
>
> if (obj->import_attach)
> return -EINVAL;
LGTM
Reviewed-by: Carl Vanderlip <quic_carlv@...cinc.com>
Powered by blists - more mailing lists