Commit d48e366e authored by Wei Yongjun's avatar Wei Yongjun Committed by Herbert Xu

crypto: tegra-aes - fix error-valued pointer dereference

clk_put(dd->aes_clk) will dereference an error-valued pointer since the
dd->aes_clk is a ERR_PTR() value. The correct check is call clk_put()
if !IS_ERR(dd->aes_clk).

dpatch engine is used to auto generate this patch.
(https://github.com/weiyj/dpatch)
Signed-off-by: default avatarWei Yongjun <yongjun_wei@trendmicro.com.cn>
Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
parent 3200da8d
...@@ -1035,7 +1035,7 @@ static int tegra_aes_probe(struct platform_device *pdev) ...@@ -1035,7 +1035,7 @@ static int tegra_aes_probe(struct platform_device *pdev)
if (dd->buf_out) if (dd->buf_out)
dma_free_coherent(dev, AES_HW_DMA_BUFFER_SIZE_BYTES, dma_free_coherent(dev, AES_HW_DMA_BUFFER_SIZE_BYTES,
dd->buf_out, dd->dma_buf_out); dd->buf_out, dd->dma_buf_out);
if (IS_ERR(dd->aes_clk)) if (!IS_ERR(dd->aes_clk))
clk_put(dd->aes_clk); clk_put(dd->aes_clk);
if (aes_wq) if (aes_wq)
destroy_workqueue(aes_wq); destroy_workqueue(aes_wq);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment