Tom Lendacky <thomas.lendacky@xxxxxxx> wrote: > Should the PSP initialization fail, the PSP data structure will be > freed and the value contained in the sp_device struct set to NULL. > At module unload, psp_dev_destroy() does not check if the pointer > value is NULL and will end up dereferencing a NULL pointer. > > Add a pointer check of the psp_data field in the sp_device struct > in psp_dev_destroy() and return immediately if it is NULL. > > Cc: <stable@xxxxxxxxxxxxxxx> # 4.16.x- > Fixes: 2a6170dfe755 ("crypto: ccp: Add Platform Security Processor (PSP) device support") > Signed-off-by: Tom Lendacky <thomas.lendacky@xxxxxxx> Patch applied. Thanks. -- Email: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt