2023-05-02 17:30 GMT+09:00, 張智諺 <cc85nod@xxxxxxxxx>:
> Sorry, I missed to add the curly brackets, I will resend new commit.
This issue was reported and it's already been fixed.
Thanks.
>
> Pumpkin <cc85nod@xxxxxxxxx> 於 2023年5月2日 週二 下午4:03寫道：
>
>> We do not assign path for those shares which have KSMBD_SHARE_FLAG_PIPE
>> flag, so we need to check if path is NULL before use it.
>>
>> [ 8.067958]
>> ==================================================================
>> [ 8.068413] BUG: KASAN: null-ptr-deref in strlen+0xa/0x40
>> [ 8.068747] Read of size 1 at addr 0000000000000000 by task
>> kworker/0:2/45
>> ...
>> [ 8.070638] Call Trace:
>> [ 8.070797] <TASK>
>> [ 8.070931] dump_stack_lvl+0x33/0x50
>> [ 8.071351] kasan_report+0xae/0xe0
>> [ 8.071766] strlen+0xa/0x40
>> [ 8.071948] getname_kernel+0x10/0x1a0
>> [ 8.072182] kern_path+0x10/0x40
>> [ 8.072385] smb2_get_info_filesystem+0xf9/0xc70
>> [ 8.076519] smb2_query_info+0x36b/0x1fd0
>> [ 8.079913] handle_ksmbd_work+0x274/0x810
>> [ 8.080165] process_one_work+0x419/0x760
>> [ 8.080421] worker_thread+0x2a2/0x6f0
>> [ 8.080916] kthread+0x160/0x190
>> [ 8.081423] ret_from_fork+0x1f/0x30
>> [ 8.081648] </TASK>
>>
>> Signed-off-by: Pumpkin <cc85nod@xxxxxxxxx>
>> ---
>> fs/ksmbd/smb2pdu.c | 3 +++
>> 1 file changed, 3 insertions(+)
>>
>> diff --git a/fs/ksmbd/smb2pdu.c b/fs/ksmbd/smb2pdu.c
>> index aa823b13d..798b193cf 100644
>> --- a/fs/ksmbd/smb2pdu.c
>> +++ b/fs/ksmbd/smb2pdu.c
>> @@ -4906,6 +4906,9 @@ static int smb2_get_info_filesystem(struct
>> ksmbd_work *work,
>> int rc = 0, len;
>> int fs_infoclass_size = 0;
>>
>> + if (share->path == NULL)
>> + return -EIO;
>> +
>> rc = kern_path(share->path, LOOKUP_NO_SYMLINKS, &path);
>> if (rc) {
>> pr_err("cannot create vfs path\n");
>> --
>> 2.39.2 (Apple Git-143)
>>
>>
>
