Re: [PATCH] dmaengine: virt-dma: Fix access after free in vcna_complete()

From: Vinod Koul
Date: Mon Dec 23 2019 - 02:54:57 EST


On 20-12-19, 16:50, Peter Ujfalusi wrote:
>
>
> On 20/12/2019 16.01, Ardelean, Alexandru wrote:
> > On Fri, 2019-12-20 at 15:11 +0200, Peter Ujfalusi wrote:
> >> [External]
> >>
> >> vchan_vdesc_fini() is freeing up 'vd' so the access to vd->tx_result is
> >> via already freed up memory.
> >>
> >> Move the vchan_vdesc_fini() after invoking the callback to avoid this.
> >>
> >
> > Apologies for seeing this too late: typo in title vcna_complete() ->
> > vchan_complete()
>
> Yep, I also noticed after sending it, I hope Vinod is kind enough and
> fix it up when applying ;)

In case it wasnt clear, yeah trivial changes while applying are no
hassle :)

--
~Vinod