- 08 Jul, 2021 2 commits
-
-
Namjae Jeon authored
If st->status is not SMB_DIRECT_CS_CONNECTED, It will jump done label and accessing the uninitialized send_ctxi by smb_direct_flush_send_list will cause kernel oops. This patch just return -ENOTCONN to avoid it. Reported-by:
Coverity Scan <scan-admin@coverity.com> Signed-off-by:
Namjae Jeon <namjae.jeon@samsung.com> Signed-off-by:
Steve French <stfrench@microsoft.com>
-
Namjae Jeon authored
netdev can never be NULL in for_each_netdev loop. This patch remove unneeded NULL check. Reported-by:
-
- 06 Jul, 2021 1 commit
-
-
Colin Ian King authored
There is a error handling case that passes control to label out_err without pointer sess being assigned a value. The unassigned pointer may be any garbage value and so the test of rc < 0 && sess maybe true leading to sess being passed to the call to ksmbd_session_destroy. Fix this by setting sess to NULL in this corner case. Addresses-Coverity: ("Uninitialized pointer read") Signed-off-by:Colin Ian King <colin.king@canonical.com> Signed-off-by:
-
- 05 Jul, 2021 1 commit
-
-
Hyunchul Lee authored
Avoid calling mnt_user_ns() many time in a function. Cc: Christoph Hellwig <hch@infradead.org> Cc: Christian Brauner <christian@brauner.io> Signed-off-by:
Hyunchul Lee <hyc.lee@gmail.com> Signed-off-by:
-
- 02 Jul, 2021 3 commits
-
-
Namjae Jeon authored
"ksmbd: remove macros in transport_ipc.c" commit change msg to req in ksmbd_rpc_ioctl/rap(). This will cause kernel oops when running smbclient -L test. Signed-off-by:
-
Hyunchul Lee authored
For user namespace support, call vfs functions with struct user_namespace got from struct path. This patch have been tested mannually as below. Create an id-mapped mount using the mount-idmapped utility (https://github.com/brauner/mount-idmapped). $ mount-idmapped --map-mount b:1003:1002:1 /home/foo <EXPORT DIR>/foo (the user, "foo" is 1003, and the user "bar" is 1002). And mount the export directory using cifs with the user, "bar". succeed to create/delete/stat/read/write files and directory in the <EXPORT DIR>/foo. But fail with a bind mount for /home/foo. Reviewed-by:
Christoph Hellwig <hch@lst.de> Signed-off-by:
-
Hyunchul Lee authored
For user namespace support, we need to pass struct user_namespace with struct dentry to some functions. For reducing the number of arguments, replace the struct dentry with struct path in these functions. Reviewed-by:
-
- 30 Jun, 2021 6 commits
-
-
Namjae Jeon authored
Reorder and document on-disk and netlink structures in headers. This is a userspace ABI to communicate data between ksmbd and user IPC daemon using netlink. This is added to track and cache user account DB and share configuration info from userspace. - KSMBD_EVENT_HEARTBEAT_REQUEST(ksmbd_heartbeat) This event is to check whether user IPC daemon is alive. If user IPC daemon is dead, ksmbd keep existing connection till disconnecting and new connection will be denied. - KSMBD_EVENT_STARTING_UP(ksmbd_startup_request) This event is to receive the information that initializes the ksmbd server from the user IPC daemon and to start the server. The global section parameters are given from smb.conf as initialization information. - KSMBD_EVENT_SHUTTING_DOWN(ksmbd_shutdown_request) This event is to shutdown ksmbd server. - KSMBD_EVENT_LOGIN_REQUEST/RESPONSE(ksmbd_login_request/response) This event is to get user account info to user IPC daemon. - KSMBD_EVENT_SHARE_CONFIG_REQUEST/RESPONSE (ksmbd_share_config_request/response) This event is to get net share configuration info. - KSMBD_EVENT_TREE_CONNECT_REQUEST/RESPONSE (ksmbd_tree_connect_request/response) This event is to get session and tree connect info. - KSMBD_EVENT_TREE_DISCONNECT_REQUEST(ksmbd_tree_disconnect_request) This event is to send tree disconnect info to user IPC daemon. - KSMBD_EVENT_LOGOUT_REQUEST(ksmbd_logout_request) This event is to send logout request to user IPC daemon. - KSMBD_EVENT_RPC_REQUEST/RESPONSE(ksmbd_rpc_command) This event is to make DCE/RPC request like srvsvc, wkssvc, lsarpc, samr to be processed in userspace. - KSMBD_EVENT_SPNEGO_AUTHEN_REQUEST/RESPONSE (ksmbd_spnego_authen_request/response) This event is to make kerberos authentication to be processed in userspace. Reviewed-by:
-
Namjae Jeon authored
Change server config string index to enumeration. Reviewed-by:
-
Namjae Jeon authored
Change server state type macro to enumeration. Reviewed-by:
-
Namjae Jeon authored
Change sid types to enumeration. Reviewed-by:
-
Namjae Jeon authored
Change ACE types to enumeration. Reviewed-by:
-
Namjae Jeon authored
ksmbd does not support SMB1. This patch remove SMB1 oplock level macros. Reviewed-by:
-
- 29 Jun, 2021 14 commits
-
-
Namjae Jeon authored
Opencode to remove ATTR_FP macro. Reviewed-by:
-
Namjae Jeon authored
Use ksmbd_vfs_lock_parent to get stable parent dentry and remove PARENT_INODE macro. Reviewed-by:
-
Namjae Jeon authored
Opencode to remove FP_INODE macro. Reviewed-by:
-
Namjae Jeon authored
Using ->d_name can be broken due to races with rename(). So use %pd with ->d_name to print filename and In other cases, use it under ->d_lock. Reviewed-by:
-
Marios Makassikis authored
smb2_validate_credit_charge() checks the CreditCharge field in the request is valid with regards to the payload size. The current implementation rejects requests with CreditCharge = 0 and a payload < 64K, even though they should be accepted. Set CreditCharge to a minimum value of 1 to avoid rejecting such requests. This matches what samba4 does. Fixes share enumeration for jcifs-ng clients. Signed-off-by:
Marios Makassikis <mmakassikis@freebox.fr> Signed-off-by:
-
Namjae Jeon authored
ksmbd: allow PROTECTED_DACL_SECINFO and UNPROTECTED_DACL_SECINFO addition information in smb2 set info security "cifsd: Fix regression in smb2_get_info" patch cause that dacl doesn't work. windows send smb2 set info security with PROTECTED_DACL_SECINFO to control dacl. But previous patch doesn't allow it. This patch add PROTECTED_DACL_SECINFO and UNPROTECTED_DACL_SECINFO addtional information flags in check. Signed-off-by:
-
Namjae Jeon authored
replace request and respone buffer macro with inline functions. Reviewed-by:
-
Namjae Jeon authored
replace SMB_DIRECT_TRANS macro with inline function. Reviewed-by:
-
Hyunchul Lee authored
Remove and replace macros with inline functions in smb_common.h Reviewed-by:
-
Hyunchul Lee authored
Remove getting worker state macros Reviewed-by:
-
Hyunchul Lee authored
Replace PAYLOAD_HEAD with inline function. Reviewed-by:
-
Hyunchul Lee authored
Replace KSMBD_ALIGN with kernel ALIGN macro Reviewed-by:
-
Hyunchul Lee authored
Replace BUFFER_NR_PAGES with inline function Reviewed-by:
-
Hyunchul Lee authored
Remove macros in transport_ipc.c Reviewed-by:
-
- 28 Jun, 2021 12 commits
-
-
Hyunchul Lee authored
set MAY_* flags together with open flags and remove ksmbd_vfs_inode_permission(). Signed-off-by:
-
Hyunchul Lee authored
Factor out a self-contained helper to get stable parent dentry. Signed-off-by:
-
Namjae Jeon authored
Rename cifsd to ksmbd and update Sergey's mail address. Reviewed-by:
-
Namjae Jeon authored
Move fs/cifsd to fs/ksmbd and rename the remaining cifsd name to ksmbd. Reviewed-by:
-
Namjae Jeon authored
Use f_bsize in FS_SECTOR_SIZE_INFORMATION to avoid the access the block layer. Signed-off-by:
-
Namjae Jeon authored
Remove unneeded NULL check in the list iterator. And use list_for_each_entry_safe instead of list_for_each_safe. Signed-off-by:
-
Namjae Jeon authored
Use f_bsize instead of q->limits.logical_block_size. Reviewed-by:
-
Namjae Jeon authored
Change stream type macro to enumeration and move it to vfs.h. Reviewed-by:
-
Namjae Jeon authored
Add select FS_POSIX_ACL in Kconfig and then opencode posix acl functions instead of wrappers Reviewed-by:
-
Namjae Jeon authored
Factor out a self-contained helper to find sub file/dir in use. Reviewed-by:
-
Namjae Jeon authored
Opencode to avoid trivial wrappers that just make the code hard to follow. Reviewed-by:
-
Namjae Jeon authored
Use the pr_fmt built into pr_*. and use pr_err/info after removing wrapper ksmbd_err/info. Reviewed-by:
-
- 22 Jun, 2021 1 commit
-
-
Namjae Jeon authored
replace KSMBD_SHARE_CONFIG_PATH with inline function. Reviewed-by:
-
