In the Linux kernel, the following vulnerability has been resolved:<\/p>\n
smb: client: fail DACL rewrite when the new DACL exceeds 64K<\/p>\n
replace_sids_and_copy_aces() and set_chmod_dacl() accumulate the size of
\nthe DACL they build in a u16. That accumulator can wrap.<\/p>\n
validate_dacl() caps num_aces at (dacl_size – sizeof(struct smb_acl)) \/
\n20, i.e. 3276 for a maximally sized DACL, while each rewritten ACE can
\ngrow to sizeof(struct smb_ace) (76 bytes) once its SID is replaced with
\none carrying SID_MAX_SUB_AUTHORITIES sub-authorities. The worst case is
\ntherefore sizeof(struct smb_acl) + 3276 * 76 = 248984 bytes, far beyond
\nwhat a u16 can hold. A wraparound is reached with 863 ACEs.<\/p>\n
After the wraparound, ndacl_ptr->size becomes meaningless and the offset
\nwill point anywhere in the ACE array. As a result, we will see
\ncorruption of the DACL, which then gets sent to the server. This is not
\nan out-of-bounds write as the allocation now covers the worst-case
\nexpansion, so writes will always go into the buffer.<\/p>\n
Adjust the code to use a u32 internally and return -EOVERFLOW in the
\noverflow case. The operation must be refused, because a DACL can only
\nhold 2^16-1 bytes on the wire and larger DACLs cannot be represented.<\/p>\n
set_chmod_dacl() carries the same pattern and is fixed the same way. It
\nonly wraps once the source DACL comes within roughly 380 bytes of the
\n64K ceiling, but the failure mode is identical.<\/p>\n","protected":false},"excerpt":{"rendered":"
In the Linux kernel, the following vulnerability has been resolved:<\/p>\n
smb: client: fail DACL rewrite when the new DACL exceeds 64K<\/p>\n
replace_sids_and_copy_aces() and set_chmod_dacl() accumulate the size<\/p>\n","protected":false},"template":"","meta":[],"categories":[],"tags":[],"class_list":["post-28991","vuln","type-vuln","status-publish","hentry"],"_links":{"self":[{"href":"https:\/\/www.54iter.cn\/index.php?rest_route=\/wp\/v2\/vuln\/28991","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/www.54iter.cn\/index.php?rest_route=\/wp\/v2\/vuln"}],"about":[{"href":"https:\/\/www.54iter.cn\/index.php?rest_route=\/wp\/v2\/types\/vuln"}],"wp:attachment":[{"href":"https:\/\/www.54iter.cn\/index.php?rest_route=%2Fwp%2Fv2%2Fmedia&parent=28991"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/www.54iter.cn\/index.php?rest_route=%2Fwp%2Fv2%2Fcategories&post=28991"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/www.54iter.cn\/index.php?rest_route=%2Fwp%2Fv2%2Ftags&post=28991"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}