2021-10-29 4:01 GMT+09:00, Marios Makassikis <mmakassikis@xxxxxxxxxx>: > The validate_negotiate_info_req struct definition includes an extra > field to access the data coming after the header. This causes the check > in fsctl_validate_negotiate_info() to count the first element of the > array twice. This in turn makes some valid requests fail, depending on > whether they include padding or not. > > Fixes: f7db8fd03a4b ("ksmbd: add validation in smb2_ioctl") > Signed-off-by: Marios Makassikis <mmakassikis@xxxxxxxxxx> Acked-by: Namjae Jeon <linkinjeon@xxxxxxxxxx> Thanks for your patch!