diff options
| author | Sabrina Dubroca <[email protected]> | 2023-10-09 20:50:42 +0000 |
|---|---|---|
| committer | David S. Miller <[email protected]> | 2023-10-13 10:26:09 +0000 |
| commit | 8f1d532b4a49e196696b0aa150962d7ce96985e4 (patch) | |
| tree | ae8b7bba910851b9e560e9294d2f97f562e48c81 /net/tls/tls_device.c | |
| parent | tls: get salt using crypto_info_salt in tls_enc_skb (diff) | |
| download | kernel-8f1d532b4a49e196696b0aa150962d7ce96985e4.tar.gz kernel-8f1d532b4a49e196696b0aa150962d7ce96985e4.zip | |
tls: drop unnecessary cipher_type checks in tls offload
We should never reach tls_device_reencrypt, tls_enc_record, or
tls_enc_skb with a cipher_type that can't be offloaded. Replace those
checks with a DEBUG_NET_WARN_ON_ONCE, and use cipher_desc instead of
hard-coding offloadable cipher types.
Signed-off-by: Sabrina Dubroca <[email protected]>
Signed-off-by: David S. Miller <[email protected]>
Diffstat (limited to 'net/tls/tls_device.c')
| -rw-r--r-- | net/tls/tls_device.c | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/net/tls/tls_device.c b/net/tls/tls_device.c index 8c94c926606a..fbd687a0c66f 100644 --- a/net/tls/tls_device.c +++ b/net/tls/tls_device.c @@ -891,14 +891,8 @@ tls_device_reencrypt(struct sock *sk, struct tls_context *tls_ctx) struct strp_msg *rxm; char *orig_buf, *buf; - switch (tls_ctx->crypto_recv.info.cipher_type) { - case TLS_CIPHER_AES_GCM_128: - case TLS_CIPHER_AES_GCM_256: - break; - default: - return -EINVAL; - } cipher_desc = get_cipher_desc(tls_ctx->crypto_recv.info.cipher_type); + DEBUG_NET_WARN_ON_ONCE(!cipher_desc || !cipher_desc->offloadable); rxm = strp_msg(tls_strp_msg(sw_ctx)); orig_buf = kmalloc(rxm->full_len + TLS_HEADER_SIZE + cipher_desc->iv, |