cifsd: simplify error handling in ksmbd_gen_preauth_integrity_hash()
authorNamjae Jeon <namjae.jeon@samsung.com>
Wed, 26 May 2021 06:56:18 +0000 (15:56 +0900)
committerNamjae Jeon <namjae.jeon@samsung.com>
Wed, 26 May 2021 09:13:07 +0000 (18:13 +0900)
Simplify error handling in ksmbd_gen_preauth_integrity_hash().

Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Namjae Jeon <namjae.jeon@samsung.com>
Signed-off-by: Steve French <stfrench@microsoft.com>
fs/cifsd/auth.c

index b0a9e45..1d4c4e6 100644 (file)
@@ -1063,14 +1063,13 @@ int ksmbd_gen_preauth_integrity_hash(struct ksmbd_conn *conn, char *buf,
        int msg_size = be32_to_cpu(rcv_hdr->smb2_buf_length);
        struct ksmbd_crypto_ctx *ctx = NULL;
 
-       if (conn->preauth_info->Preauth_HashId ==
-           SMB2_PREAUTH_INTEGRITY_SHA512) {
-               ctx = ksmbd_crypto_ctx_find_sha512();
-               if (!ctx) {
-                       ksmbd_debug(AUTH, "could not alloc sha512 rc %d\n", rc);
-                       goto out;
-               }
-       } else {
+       if (conn->preauth_info->Preauth_HashId !=
+           SMB2_PREAUTH_INTEGRITY_SHA512)
+               return -EINVAL;
+
+       ctx = ksmbd_crypto_ctx_find_sha512();
+       if (!ctx) {
+               ksmbd_debug(AUTH, "could not alloc sha512 rc %d\n", rc);
                goto out;
        }