diff options
| author |   Eric Dumazet <eric.dumazet@gmail.com> | 2011-10-21 05:22:42 -0400 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2011-10-21 05:22:42 -0400 |
| commit | cf533ea53ebfae41be15b103d78e7ebec30b9969 (patch) | |
| tree | 51ed3c69f4a15117fefe5cbd291a75010beb0f4b /include/net/secure_seq.h | |
| parent | dev: use name hash for dev_seq_ops (diff) | |
| download | wireguard-linux-cf533ea53ebfae41be15b103d78e7ebec30b9969.tar.xz wireguard-linux-cf533ea53ebfae41be15b103d78e7ebec30b9969.zip | |
tcp: add const qualifiers where possible
Adding const qualifiers to pointers can ease code review, and spot some
bugs. It might allow compiler to optimize code further.
For example, is it legal to temporary write a null cksum into tcphdr
in tcp_md5_hash_header() ? I am afraid a sniffer could catch the
temporary null value...
Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/net/secure_seq.h')
| -rw-r--r-- | include/net/secure_seq.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/net/secure_seq.h b/include/net/secure_seq.h index d97f6892c019..c2e542b27a5a 100644 --- a/include/net/secure_seq.h +++ b/include/net/secure_seq.h @@ -10,7 +10,7 @@ extern u32 secure_ipv6_port_ephemeral(const __be32 *saddr, const __be32 *daddr, __be16 dport); extern __u32 secure_tcp_sequence_number(__be32 saddr, __be32 daddr, __be16 sport, __be16 dport); -extern __u32 secure_tcpv6_sequence_number(__be32 *saddr, __be32 *daddr, +extern __u32 secure_tcpv6_sequence_number(const __be32 *saddr, const __be32 *daddr, __be16 sport, __be16 dport); extern u64 secure_dccp_sequence_number(__be32 saddr, __be32 daddr, __be16 sport, __be16 dport); |