Commit bc76e5bb authored by Jakub Kicinski's avatar Jakub Kicinski Committed by David S. Miller
Browse files

net/tls: store decrypted on a single bit 



Use a single bit instead of boolean to remember if packet
was already decrypted.

Signed-off-by: default avatarJakub Kicinski <jakub.kicinski@netronome.com>
Reviewed-by: default avatarSimon Horman <simon.horman@netronome.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 5c5458ec

include/net/tls.h

+1 −1
Original line number Diff line number Diff line
@@ -153,7 +153,7 @@ struct tls_sw_context_rx { 
	struct sk_buff *recv_pkt;

	u8 control;

	u8 async_capable:1;

	bool decrypted;

	u8 decrypted:1;

	atomic_t decrypt_pending;

	bool async_notify;

};

net/tls/tls_sw.c

+3 −3
Original line number Diff line number Diff line
@@ -1523,7 +1523,7 @@ static int decrypt_skb_update(struct sock *sk, struct sk_buff *skb, 
		rxm->offset += prot->prepend_size;

		rxm->full_len -= prot->overhead_size;

		tls_advance_record_sn(sk, prot, &tls_ctx->rx);

		ctx->decrypted = true;

		ctx->decrypted = 1;

		ctx->saved_data_ready(sk);

	} else {

		*zc = false;
@@ -1933,7 +1933,7 @@ ssize_t tls_sw_splice_read(struct socket *sock, loff_t *ppos, 
			tls_err_abort(sk, EBADMSG);

			goto splice_read_end;

		}

		ctx->decrypted = true;

		ctx->decrypted = 1;

	}

	rxm = strp_msg(skb);
@@ -2034,7 +2034,7 @@ static void tls_queue(struct strparser *strp, struct sk_buff *skb) 
	struct tls_context *tls_ctx = tls_get_ctx(strp->sk);

	struct tls_sw_context_rx *ctx = tls_sw_ctx_rx(tls_ctx);



	ctx->decrypted = false;

	ctx->decrypted = 0;



	ctx->recv_pkt = skb;

	strp_pause(strp);

CRA Git | Maintained and supported by SUSTech CRA and CCSE