原文

Linuxカーネルにおいて、以下の脆弱性が修正されました。ksmbd: SMB2 SET_INFO SECURITYにWRITE_DACおよびWRITE_OWNERチェックを追加しました。コミットcc57232cae23（"ksmbd: fix FSCTL permission bypass by adding a permission check for FSCTL_SET_SPARSE"）では、fsctl_set_sparseにfp-＞daccessゲートを追加し、「類似のハンドルレベルのチェックは他の関数に存在するが、ここでは欠如している」と指摘されています。SMB2 SET_INFO SECURITYの部分はその欠如している一つであり、最もセキュリティに関連が高い部分です。smb2_set_info_sec()関数はハンドルごとのアクセスチェックを行わずにset_info_sec()を呼び出していました。set_info_sec()（fs/smb/server/smbacl.c）はファイルの権限を再設定します。notify_change()を通じて所有者・グループ・モードを書き換え、set_posix_acl()でPOSIX ACLを再設定し、KSMBD_SHARE_FLAG_ACL_XATTR共有上ではksmbd_vfs_set_sd_xattr()によってWindowsのセキュリティ記述子を削除・再設定します。兄弟ハンドラであるsmb2_set_info_file()の他の永続的な変更を行う部分はいずれも先にfp-＞daccessの確認を行っています（FILE_WRITE_DATA / FILE_DELETE / FILE_WRITE_EA / FILE_WRITE_ATTRIBUTES）。しかし、アクセス制御自体を変更するSECURITY部分のみがゲートチェックを持っていませんでした。そのためクライアントはFILE_WRITE_ATTRIBUTESのみ（FILE_WRITE_DACやFILE_WRITE_OWNERは含まない）でハンドルを開き、SMB2_SET_INFOでInfoTypeとしてSMB2_O_INFO_SECURITYを使うことで、ファイルのDACLと所有者を上書きし、自身にハンドルのdaccessに存在しなかったアクセス権を付与することが可能でした。FSCTLのデータ操作部とは異なり、これはメタデータやxattrの操作であるため、FMODE_WRITEによるVFSのバックストップは存在せず、欠如していたfp-＞daccessチェックのみが唯一のゲートとなっていました。セキュリティ記述子の設定はWRITE_DACおよびWRITE_OWNER操作に該当するため、ファイルの権限を書き換える前にハンドルにこれらいずれかの権限があることを要求するようにしました。-EACCESはsmb2_set_info()によってSTATUS_ACCESS_DENIEDにマッピングされます。