On Wed, Jul 20, 2022 at 8:29 AM Namburu, Chandu-babu <cha...@amd.com> wrote:
> From: Shiqi Liu <835703...@qq.com>
>
> As the possible failure of the rte_malloc(), the not_checked and checked
> could be NULL pointer.
> Therefore, it should be better to check it in order to avoid the dereference
> of the NULL pointer.
>
> Fixes: 09a0fd736a0 ("crypto/ccp: enable IOMMU")
> Signed-off-by: Shiqi Liu <835703...@qq.com>
This sha_ctx variable and its accesses are suspicious.
It seems to be used as some kind of intermediate buffer, but I fail to
see the need.
Can't the existing code rely on sess->auth.ctx ?
There is also a suspicious mention (in ccp_perform_sha) of sha_ctx but
with no calling rte_mem_virt2iova().
--
David Marchand
