diff options
| author | Alexander Duyck <aduyck@mirantis.com> | 2016-03-11 14:05:41 -0800 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2016-03-13 23:55:13 -0400 |
| commit | 1e94082963747b551b129528714827f76a090e93 (patch) | |
| tree | 543178f26d73ba0a7c9307091200ba6856c9fd60 /net/ipv6 | |
| parent | 01cfbad79a5e2b835abf6a8154a341d75a6fc8cd (diff) | |
| download | lwn-1e94082963747b551b129528714827f76a090e93.tar.gz lwn-1e94082963747b551b129528714827f76a090e93.zip | |
ipv6: Pass proto to csum_ipv6_magic as __u8 instead of unsigned short
This patch updates csum_ipv6_magic so that it correctly recognizes that
protocol is a unsigned 8 bit value.
This will allow us to better understand what limitations may or may not be
present in how we handle the data. For example there are a number of
places that call htonl on the protocol value. This is likely not necessary
and can be replaced with a multiplication by ntohl(1) which will be
converted to a shift by the compiler.
Signed-off-by: Alexander Duyck <aduyck@mirantis.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv6')
| -rw-r--r-- | net/ipv6/ip6_checksum.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/net/ipv6/ip6_checksum.c b/net/ipv6/ip6_checksum.c index 8f920580976f..b2025bf3da4a 100644 --- a/net/ipv6/ip6_checksum.c +++ b/net/ipv6/ip6_checksum.c @@ -6,8 +6,7 @@ #ifndef _HAVE_ARCH_IPV6_CSUM __sum16 csum_ipv6_magic(const struct in6_addr *saddr, const struct in6_addr *daddr, - __u32 len, unsigned short proto, - __wsum csum) + __u32 len, __u8 proto, __wsum csum) { int carry; |