ksmbd: free ppace array on error in parse_dacl
authorFedor Pchelkin <pchelkin@ispras.ru>
Tue, 9 Jan 2024 14:14:44 +0000 (17:14 +0300)
committerSteve French <stfrench@microsoft.com>
Wed, 10 Jan 2024 01:27:36 +0000 (19:27 -0600)
commit8cf9bedfc3c47d24bb0de386f808f925dc52863e
treee7c8db7e9ca26fd266cc040117015fa3886ca0e8
parent3fc74c65b367476874da5fe6f633398674b78e5a
ksmbd: free ppace array on error in parse_dacl

The ppace array is not freed if one of the init_acl_state() calls inside
parse_dacl() fails. At the moment the function may fail only due to the
memory allocation errors so it's highly unlikely in this case but
nevertheless a fix is needed.

Move ppace allocation after the init_acl_state() calls with proper error
handling.

Found by Linux Verification Center (linuxtesting.org).

Fixes: e2f34481b24d ("cifsd: add server-side procedures for SMB3")
Cc: stable@vger.kernel.org
Signed-off-by: Fedor Pchelkin <pchelkin@ispras.ru>
Acked-by: Namjae Jeon <linkinjeon@kernel.org>
Signed-off-by: Steve French <stfrench@microsoft.com>
fs/smb/server/smbacl.c