[PATCH v3 5/6] drm/i915/gvt: Assign NULL to the pointer after memory free.
Zhenyu Wang
zhenyuw at linux.intel.com
Tue Apr 30 02:39:01 UTC 2019
On 2019.04.29 12:22:06 +0000, Gimbitskii, Aleksei wrote:
> Thanks for the reminding and sorry that I forget to mention that. But we think that if we add this error message here, probably we also need to add the error message in other similar places, which seems a bit over too much. How about we still follow the existing style?
>
Then instead of use-after-free, just remove that error message,
and provide cleaner error message in future if helpful.
> -----Original Message-----
> From: Xu, Colin
> Sent: Thursday, April 25, 2019 8:32 AM
> To: Gimbitskii, Aleksei <aleksei.gimbitskii at intel.com>
> Cc: Zhenyu Wang <zhenyuw at linux.intel.com>; intel-gvt-dev at lists.freedesktop.org; Wang, Zhi A <zhi.a.wang at intel.com>
> Subject: Re: [PATCH v3 5/6] drm/i915/gvt: Assign NULL to the pointer after memory free.
>
>
> On 2019-04-25 11:14, Zhenyu Wang wrote:
> > On 2019.04.23 15:04:12 +0300, 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 061499a38f4d..927753a59a1e
> >> 100644
> >> --- a/drivers/gpu/drm/i915/gvt/gtt.c
> >> +++ b/drivers/gpu/drm/i915/gvt/gtt.c
> >> @@ -1106,6 +1106,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);
> > I think we can remove this error message which doesn't tell the reason
> > to fail at all, but have err message in earlier path where we can
> > indicate the reason.
>
> Seem like miss my comment in v2.
>
> I would suggest add different msg before the two "goto err_free_spt",
>
> then in this err_free_stp label you could just return without print the
>
> same msg as err. Otherwise you can't tell the reason from the same err.
>
> >
> > _______________________________________________
> > intel-gvt-dev mailing list
> > intel-gvt-dev at lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/intel-gvt-dev
>
> --
> Best Regards,
> Colin Xu
>
--
Open Source Technology Center, Intel ltd.
$gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/intel-gvt-dev/attachments/20190430/677fe7cb/attachment.sig>
More information about the intel-gvt-dev
mailing list