diff options
| author | Namjae Jeon <linkinjeon@kernel.org> | 2026-07-04 11:07:51 +0900 |
|---|---|---|
| committer | Steve French <stfrench@microsoft.com> | 2026-07-22 09:54:10 -0500 |
| commit | 5152c6d49e3fd4e9f2e857c57527aead752f1f87 (patch) | |
| tree | b2187df63b539c9002e79b35dc84fb8a2da87e1e /scripts | |
| parent | bbf0a8e931204ecdab494a88d43b0a24a04285c5 (diff) | |
ksmbd: validate ACE size against SID sub-authorities
set_ntacl_dacl() validates sid.num_subauth before copying an ACE, but
does not verify that the declared ACE size contains all sub-authorities
described by that field. An undersized ACE can therefore be copied
and later make the POSIX ACL deduplication walk inspect data beyond
the copied ACE boundary.
The existing initial bound check is also too small. It only ensures
that the ACE size field is accessible before set_ntacl_dacl() reads
sid.num_subauth farther into the input buffer.
Require enough input for the fixed SID header before accessing
num_subauth, reject ACEs smaller than that header, and skip ACEs
whose declared size cannot contain the complete SID. This makes the
validation consistent with the other ACE walk paths.
Reported-by: LocalHost <localhost.detect@gmail.com>
Signed-off-by: Namjae Jeon <linkinjeon@kernel.org>
Signed-off-by: Steve French <stfrench@microsoft.com>
Diffstat (limited to 'scripts')
0 files changed, 0 insertions, 0 deletions
