ksmbd: Fix potential memory leak in tcp_destroy_socket()

ksmbd_socket must be freed even if kernel_sock_shutdown() somehow fails.

Signed-off-by: Marios Makassikis <mmakassikis@freebox.fr>
Signed-off-by: Namjae Jeon <namjae.jeon@samsung.com>
Signed-off-by: Steve French <stfrench@microsoft.com>

diff --git a/fs/ksmbd/transport_tcp.c b/fs/ksmbd/transport_tcp.c
index 56ec11ff5a9f5b0a3ef1717339e68975a9d9ff80..dc15a5ecd2e028daa56bc6272021d454ac111ce5 100644 (file)
--- a/fs/ksmbd/transport_tcp.c
+++ b/fs/ksmbd/transport_tcp.c
@@ -381,8 +381,7 @@ static void tcp_destroy_socket(struct socket *ksmbd_socket)
        ret = kernel_sock_shutdown(ksmbd_socket, SHUT_RDWR);
        if (ret)
                pr_err("Failed to shutdown socket: %d\n", ret);
-       else
-               sock_release(ksmbd_socket);
+       sock_release(ksmbd_socket);
 }
 
 /**