Commit f8d15d29 authored by Ayush Sawal's avatar Ayush Sawal Committed by Jakub Kicinski
Browse files

chtls: Avoid unnecessary freeing of oreq pointer 



In chtls_pass_accept_request(), removing the chtls_reqsk_free()
call to avoid oreq freeing twice. Here oreq is the pointer to
struct request_sock.

Fixes: cc35c88a ("crypto : chtls - CPL handler definition")
Signed-off-by: default avatarRohit Maheshwari <rohitm@chelsio.com>
Signed-off-by: default avatarAyush Sawal <ayush.sawal@chelsio.com>
Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent 5a5fac99

drivers/net/ethernet/chelsio/inline_crypto/chtls/chtls_cm.c

+1 −3
Original line number Diff line number Diff line
@@ -1397,7 +1397,7 @@ static void chtls_pass_accept_request(struct sock *sk, 


	newsk = chtls_recv_sock(sk, oreq, network_hdr, req, cdev);

	if (!newsk)

		goto free_oreq;

		goto reject;



	if (chtls_get_module(newsk))

		goto reject;
@@ -1413,8 +1413,6 @@ static void chtls_pass_accept_request(struct sock *sk, 
	kfree_skb(skb);

	return;



free_oreq:

	chtls_reqsk_free(oreq);

reject:

	mk_tid_release(reply_skb, 0, tid);

	cxgb4_ofld_send(cdev->lldi->ports[0], reply_skb);