Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Private/marios/smb1 next #458

Closed

Conversation

mmakassikis
Copy link

Some fixes from testing with smbtorture

Marios Makassikis added 7 commits October 12, 2023 19:19
automatic changes using clang-format to:
 * keep code around ~80 columns
 * fix some odd argument indentations
 * remove double spaces

Not all changes were picked as the clang-formatted code is not always
more readable.

This makes the code a bit more coherent and brings it closer to the
Linux kernel code style.

while here fix some typos.

Signed-off-by: Marios Makassikis <[email protected]>
SMB_COM_PROCESS_EXIT is a deprecated command from earlier CIFS dialects
that is really only used by smbtorture tests.

Signed-off-by: Marios Makassikis <[email protected]>
Fixes a memory leak in smb_get_name() and a double-free in
smb_get_dir_name() if a name with an invalid character was passed
in.

Signed-off-by: Marios Makassikis <[email protected]>
[MS-CIFS] 3.3.5.33
  * If EchoCount is zero, a response MUST NOT be sent
  * The server MAY enforce any nonzero limit in the number of responses
    that it returns

Signed-off-by: Marios Makassikis <[email protected]>
SMB_COM_QUERY_INFORMATION_DISK is marked as deprecated, but smbclient
will use it if SMB Trans2/QUERY_FS_INFO fails. ksmbd will disconnect
the client because the command is not implemented.

The response to this command contains the same information as
QUERY_FS_SIZE_INFO/QUERY_FS_FULL_SIZE_INFO infolevels. The difference is
that the fields are u16, so they may not be large enough. Values are
adjusted so that the client can determine free and used space in bytes
(which is what it really cares about).

Signed-off-by: Marios Makassikis <[email protected]>
@namjaejeon
Copy link
Owner

Applied! Thansk for your patch!

@namjaejeon namjaejeon closed this Oct 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants