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