Re: [PATCH 2/3] ksmbd: fix partial normalized name responses

[ChenXiaoSong <chenxiaosong@chenxiaosong.com>]

If needed, I would be happy to help send v2.

And it would be better to use FILE_NAME_INFORMATION_SIZE (should be 
changed to 8) instead of FILE_ALTERNATE_NAME_INFORMATION_SIZE.

```
smb2_get_info_file()
{
...
	case FILE_ALTERNATE_NAME_INFORMATION:
         case FILE_NORMALIZED_NAME_INFORMATION:
                 fixed_len = FILE_NAME_INFORMATION_SIZE; // 8
...
}
```

On 9/14/26 10:38, ChenXiaoSong wrote:
> Hi Namjae,
> 
> According to MS-FSA 2.1.5.12.30: https://learn.microsoft.com/en-us/ 
> openspecs/windows_protocols/ms-fsa/3d5c68f8-fbc5-4f94-a51d-3600a319fd45
> 
>    Set AvailableNameLength to BlockAlignTruncate((OutputBufferSize - 
> FieldOffset(FILE_NAME_INFORMATION.FileName)), 2).
> 
> It seems that the following changes are also needed:
> ```
> smb2_get_info_file()
> {
>          unsigned int req_output_len = le32_to_cpu(req- 
>  >OutputBufferLength);
> ...
>          case FILE_NORMALIZED_NAME_INFORMATION:
>                  fixed_len = FILE_NORMALIZED_NAME_INFORMATION_SIZE;
>                  req_output_len = round_down(req_output_len, 2);
>                  break;
> ...
>          rc = buffer_check_err(req_output_len, fixed_len, rsp);
> }
> ```
> 
> By the way, it seems that `struct smb2_file_alt_name_info` should be 
> renamed to `struct smb2_file_name_info`, since both 
> FILE_ALTERNATE_NAME_INFORMATION and FILE_NORMALIZED_NAME_INFORMATION use 
> this structure. See:
> - MS-FSA 2.1.5.12.4 FileAlternateNameInformation: https:// 
> learn.microsoft.com/en-us/openspecs/windows_protocols/ms- 
> fsa/5051d021-8d06-4b7e-94e3-55b118193427
> - MS-FSA 2.1.5.12.30 FileNormalizedNameInformation: https:// 
> learn.microsoft.com/en-us/openspecs/windows_protocols/ms-fsa/3d5c68f8- 
> fbc5-4f94-a51d-3600a319fd45

-- 
ChenXiaoSong <chenxiaosong@chenxiaosong.com>
Chinese Homepage: https://chenxiaosong.com
English Homepage: https://chenxiaosong.com/en