aboutsummaryrefslogtreecommitdiffstats
path: root/net/tls/tls_main.c
diff options
context:
space:
mode:
authorBoris Pismenny <borisp@mellanox.com>2018-02-14 03:46:07 -0500
committerDavid S. Miller <davem@davemloft.net>2018-02-14 15:05:19 -0500
commit257082e6ae23e92898440f6bcb2857555bf7957c (patch)
treefd325d7e2199022ad42b2f7fa8a0b83c397cf835 /net/tls/tls_main.c
parenta1dfa6812b682eef750412dd5a90e7d38d7af068 (diff)
tls: reset the crypto info if copy_from_user fails
copy_from_user could copy some partial information, as a result TLS_CRYPTO_INFO_READY(crypto_info) could be true while crypto_info is using uninitialzed data. This patch resets crypto_info when copy_from_user fails. fixes: 3c4d7559159b ("tls: kernel TLS support") Signed-off-by: Boris Pismenny <borisp@mellanox.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/tls/tls_main.c')
-rw-r--r--net/tls/tls_main.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/net/tls/tls_main.c b/net/tls/tls_main.c
index a6c3702e0ddb..c105f86a7ea6 100644
--- a/net/tls/tls_main.c
+++ b/net/tls/tls_main.c
@@ -376,7 +376,7 @@ static int do_tls_setsockopt_tx(struct sock *sk, char __user *optval,
376 rc = copy_from_user(crypto_info, optval, sizeof(*crypto_info)); 376 rc = copy_from_user(crypto_info, optval, sizeof(*crypto_info));
377 if (rc) { 377 if (rc) {
378 rc = -EFAULT; 378 rc = -EFAULT;
379 goto out; 379 goto err_crypto_info;
380 } 380 }
381 381
382 /* check version */ 382 /* check version */