On Tue, Feb 25, 2020 at 1:06 PM Rijo Thomas Rijo-john.Thomas@amd.com 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@oracle.com
Reviewed-by: Rijo Thomas Rijo-john.Thomas@amd.com
I'm picking up this.
Thanks, Jens