Re: [PATCH v2] tee: amdtee: fix memory leak in amdtee_open_session()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Feb 25, 2020 at 1:06 PM Rijo Thomas <Rijo-john.Thomas@xxxxxxx> wrote:
>
>
>
> On 24/02/20 4:21 pm, Dan Carpenter wrote:
> > On these error paths the "sess" variable isn't freed.  It's a refcounted
> > pointer so we need to call kref_put().  I re-arranged the code a bit so
> > the error case is always handled before the success case and the error
> > paths are indented two tabs.
> >
> > Fixes: 757cc3e9ff1d ("tee: add AMD-TEE driver")
> > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
>
> Reviewed-by: Rijo Thomas <Rijo-john.Thomas@xxxxxxx>

I'm picking up this.

Thanks,
Jens



[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux