diff options
| author |   Namjae Jeon <linkinjeon@kernel.org> | 2022-09-22 23:35:43 +0900 |
|---|---|---|
| committer |   Steve French <stfrench@microsoft.com> | 2022-10-05 01:15:44 -0500 |
| commit | 360c8ee6fefdb496fffd2c18bb9a96a376a1a804 (patch) | |
| tree | 1ab57079bfeb191cde267ccf168f4611e78b8ac4 /fs/ksmbd/server.c | |
| parent | ksmbd: fill sids in SMB_FIND_FILE_POSIX_INFO response (diff) | |
| download | linux-dev-360c8ee6fefdb496fffd2c18bb9a96a376a1a804.tar.xz linux-dev-360c8ee6fefdb496fffd2c18bb9a96a376a1a804.zip | |
ksmbd: fix endless loop when encryption for response fails
If ->encrypt_resp return error, goto statement cause endless loop.
It send an error response immediately after removing it.
Fixes: 0626e6641f6b ("cifsd: add server handler for central processing and tranport layers")
Cc: stable@vger.kernel.org
Signed-off-by: Namjae Jeon <linkinjeon@kernel.org>
Signed-off-by: Steve French <stfrench@microsoft.com>
Diffstat (limited to 'fs/ksmbd/server.c')
| -rw-r--r-- | fs/ksmbd/server.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/fs/ksmbd/server.c b/fs/ksmbd/server.c index ce42bff42ef9..a0d635304754 100644 --- a/fs/ksmbd/server.c +++ b/fs/ksmbd/server.c @@ -235,10 +235,8 @@ send: if (work->sess && work->sess->enc && work->encrypted && conn->ops->encrypt_resp) { rc = conn->ops->encrypt_resp(work); - if (rc < 0) { + if (rc < 0) conn->ops->set_rsp_status(work, STATUS_DATA_ERROR); - goto send; - } } ksmbd_conn_write(work); |