diff options
| author | Dmitry Safonov <[email protected]> | 2023-12-04 19:00:41 +0000 |
|---|---|---|
| committer | Paolo Abeni <[email protected]> | 2023-12-06 11:36:55 +0000 |
| commit | da7dfaa6d6f731c30eca6ffa808b83634d43e26f (patch) | |
| tree | 6664d5827e0611238fc46e4225a8b435754d72a7 /net/ipv6/tcp_ipv6.c | |
| parent | Documentation/tcp: Fix an obvious typo (diff) | |
| download | kernel-da7dfaa6d6f731c30eca6ffa808b83634d43e26f.tar.gz kernel-da7dfaa6d6f731c30eca6ffa808b83634d43e26f.zip | |
net/tcp: Consistently align TCP-AO option in the header
Currently functions that pre-calculate TCP header options length use
unaligned TCP-AO header + MAC-length for skb reservation.
And the functions that actually write TCP-AO options into skb do align
the header. Nothing good can come out of this for ((maclen % 4) != 0).
Provide tcp_ao_len_aligned() helper and use it everywhere for TCP
header options space calculations.
Fixes: 1e03d32bea8e ("net/tcp: Add TCP-AO sign to outgoing packets")
Signed-off-by: Dmitry Safonov <[email protected]>
Reviewed-by: Eric Dumazet <[email protected]>
Signed-off-by: Paolo Abeni <[email protected]>
Diffstat (limited to 'net/ipv6/tcp_ipv6.c')
| -rw-r--r-- | net/ipv6/tcp_ipv6.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ipv6/tcp_ipv6.c b/net/ipv6/tcp_ipv6.c index 937a02c2e534..8c6623496dd7 100644 --- a/net/ipv6/tcp_ipv6.c +++ b/net/ipv6/tcp_ipv6.c @@ -881,7 +881,7 @@ static void tcp_v6_send_response(const struct sock *sk, struct sk_buff *skb, u32 if (tcp_key_is_md5(key)) tot_len += TCPOLEN_MD5SIG_ALIGNED; if (tcp_key_is_ao(key)) - tot_len += tcp_ao_len(key->ao_key); + tot_len += tcp_ao_len_aligned(key->ao_key); #ifdef CONFIG_MPTCP if (rst && !tcp_key_is_md5(key)) { |