ksmbd: use aead_request_free to match aead_request_alloc

[ Upstream commit 6171063e9d046ffa46f51579b2ca4a43caef581a ]

Use aead_request_free() instead of kfree() to properly free memory
allocated by aead_request_alloc(). This ensures sensitive crypto data
is zeroed before being freed.

Fixes: e2f34481b24d ("cifsd: add server-side procedures for SMB3")
Signed-off-by: Miaoqian Lin <linmq006@gmail.com>
Acked-by: Namjae Jeon <linkinjeon@kernel.org>
Signed-off-by: Steve French <stfrench@microsoft.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>

diff --git a/fs/smb/server/auth.c b/fs/smb/server/auth.c
index 8e24a6665abdb5b4ad1eb176e303bd924d249f72..c34b30642bfddabf695a48f45a6912215e229e27 100644 (file)
--- a/fs/smb/server/auth.c
+++ b/fs/smb/server/auth.c
@@ -1211,7 +1211,7 @@ free_iv:
 free_sg:
        kfree(sg);
 free_req:
-       kfree(req);
+       aead_request_free(req);
 free_ctx:
        ksmbd_release_crypto_ctx(ctx);
        return rc;