diff mbox series

[net-next,3/3] net/tls: Remove redundant variable assignments and wakeup

Message ID 20180713222230.17949-3-vakul.garg@nxp.com
State Superseded, archived
Delegated to: David Miller
Headers show
Series None | expand

Commit Message

Vakul Garg July 13, 2018, 10:22 p.m. UTC 
In function tls_do_decryption(), the assignment to tls receive context'
variable 'decrypted' is redundant as the same is being done in function
tls_recvmsg() after calling decrypt_skb(). Also calling callback
function to wakeup processes sleeping on socket data availability is
useless as tls_do_decryption() is invoked from user processes only. This
patch cleans these up.

Signed-off-by: Vakul Garg <vakul.garg@nxp.com>
---
 net/tls/tls_sw.c | 4 ----
 1 file changed, 4 deletions(-)
diff mbox series

Patch

diff --git a/net/tls/tls_sw.c b/net/tls/tls_sw.c
index a184c70ee0ac..a53fdcc33e31 100644
--- a/net/tls/tls_sw.c
+++ b/net/tls/tls_sw.c
@@ -78,10 +78,6 @@  static int tls_do_decryption(struct sock *sk,
 	rxm->full_len -= tls_ctx->rx.overhead_size;
 	tls_advance_record_sn(sk, &tls_ctx->rx);
 
-	ctx->decrypted = true;
-
-	ctx->saved_data_ready(sk);
-
 out:
 	aead_request_free(aead_req);
 	return ret;