[PATCH v4 2/2] drm/i915/gvt: Assign NULL to the pointer after memory free.
Colin Xu
Colin.Xu at intel.com
Tue May 21 05:16:17 UTC 2019
Acked-by: Colin Xu<colin.xu at intel.com>
I'm OK with the change. Zhenyu has some comments in v3 see if he has
more comments in v4. Thanks.
On 2019-05-02 17:59, Aleksei Gimbitskii wrote:
> The klocwork static code analyzer complains about using pointer after
> being freed, because further we pass it to the gvt_vgpu_err() function.
> Assign pointer to be NULL intentionaly, to meet requirements of the code
> analyzer.
>
> This patch fixed the issue #648 reported as error by klocwork.
>
> Signed-off-by: Aleksei Gimbitskii <aleksei.gimbitskii at intel.com>
> Cc: Zhenyu Wang <zhenyuw at linux.intel.com>
> Cc: Zhi Wang <zhi.a.wang at intel.com>
> ---
> drivers/gpu/drm/i915/gvt/gtt.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/gpu/drm/i915/gvt/gtt.c b/drivers/gpu/drm/i915/gvt/gtt.c
> index 7e61396a65c6..2c723509ab68 100644
> --- a/drivers/gpu/drm/i915/gvt/gtt.c
> +++ b/drivers/gpu/drm/i915/gvt/gtt.c
> @@ -1109,6 +1109,7 @@ static struct intel_vgpu_ppgtt_spt *ppgtt_populate_spt_by_guest_entry(
>
> err_free_spt:
> ppgtt_free_spt(spt);
> + spt = NULL;
> err:
> gvt_vgpu_err("fail: shadow page %p guest entry 0x%llx type %d\n",
> spt, we->val64, we->type);
--
Best Regards,
Colin Xu
More information about the intel-gvt-dev
mailing list