Re: [PATCH v2 2/4] ksmbd: add validation in smb2_ioctl

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



2021-09-21 17:08 GMT+09:00, Ralph Boehme <slow@xxxxxxxxx>:
> Hi Namjae,
>
>
> Am 19.09.21 um 04:13 schrieb Namjae Jeon:
>> Add validation for request/response buffer size check in smb2_ioctl.
>>
>> Cc: Ronnie Sahlberg <ronniesahlberg@xxxxxxxxx>
>> Cc: Ralph Böhme <slow@xxxxxxxxx>
>> Cc: Steve French <smfrench@xxxxxxxxx>
>> Signed-off-by: Namjae Jeon <linkinjeon@xxxxxxxxxx>
>
> the check looks good. May I suggest to also pass the pointers to the
> structs instead of the raw buffers consistently, same as we do for the
> setinfo levels now?
>
> We still pass the raw buffer to fsctl_query_iface_info_ioctl and
> fsctl_copychunk().
Hm.. Let me check it. maybe will fix it on another patch.
>
> -slow
>
> --
> Ralph Boehme, Samba Team                 https://samba.org/
> SerNet Samba Team Lead      https://sernet.de/en/team-samba
>
>




[Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux