diff options
| author | David Disseldorp | 2023-04-13 23:49:57 +0900 |
|---|---|---|
| committer | Greg Kroah-Hartman | 2023-04-20 12:35:12 +0200 |
| commit | a2f6ded41bec1d3be643c80a5eb97f1680309001 (patch) | |
| tree | d7b52dc28397cba659e0ed3c06b4e1b4d155f58b /arch/riscv/kernel | |
| parent | 2128f7c00fa5d5bca1186588f4f197faad2b4460 (diff) | |
ksmbd: avoid out of bounds access in decode_preauth_ctxt()
commit e7067a446264a7514fa1cfaa4052cdb6803bc6a2 upstream.
Confirm that the accessed pneg_ctxt->HashAlgorithms address sits within
the SMB request boundary; deassemble_neg_contexts() only checks that the
eight byte smb2_neg_context header + (client controlled) DataLength are
within the packet boundary, which is insufficient.
Checking for sizeof(struct smb2_preauth_neg_context) is overkill given
that the type currently assumes SMB311_SALT_SIZE bytes of trailing Salt.
Signed-off-by: David Disseldorp <ddiss@suse.de>
Acked-by: Namjae Jeon <linkinjeon@kernel.org>
Cc: <stable@vger.kernel.org>
Signed-off-by: Steve French <stfrench@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'arch/riscv/kernel')
0 files changed, 0 insertions, 0 deletions