diff options
-rw-r--r-- | include/net/sctp/checksum.h | 23 | ||||
-rw-r--r-- | net/ipv4/netfilter/nf_nat_proto_sctp.c | 4 | ||||
-rw-r--r-- | net/sctp/input.c | 4 | ||||
-rw-r--r-- | net/sctp/output.c | 4 |
4 files changed, 20 insertions, 15 deletions
diff --git a/include/net/sctp/checksum.h b/include/net/sctp/checksum.h index ba75c67cb992..b799fb21519a 100644 --- a/include/net/sctp/checksum.h +++ b/include/net/sctp/checksum.h | |||
@@ -46,9 +46,14 @@ | |||
46 | #include <net/sctp/sctp.h> | 46 | #include <net/sctp/sctp.h> |
47 | #include <linux/crc32c.h> | 47 | #include <linux/crc32c.h> |
48 | 48 | ||
49 | static inline __u32 sctp_start_cksum(__u8 *buffer, __u16 length) | 49 | static inline __be32 sctp_crc32c(__be32 crc, u8 *buffer, u16 length) |
50 | { | 50 | { |
51 | __u32 crc = ~(__u32) 0; | 51 | return (__force __be32)crc32c((__force u32)crc, buffer, length); |
52 | } | ||
53 | |||
54 | static inline __be32 sctp_start_cksum(__u8 *buffer, __u16 length) | ||
55 | { | ||
56 | __be32 crc = ~cpu_to_be32(0); | ||
52 | __u8 zero[sizeof(__u32)] = {0}; | 57 | __u8 zero[sizeof(__u32)] = {0}; |
53 | 58 | ||
54 | /* Optimize this routine to be SCTP specific, knowing how | 59 | /* Optimize this routine to be SCTP specific, knowing how |
@@ -56,23 +61,23 @@ static inline __u32 sctp_start_cksum(__u8 *buffer, __u16 length) | |||
56 | */ | 61 | */ |
57 | 62 | ||
58 | /* Calculate CRC up to the checksum. */ | 63 | /* Calculate CRC up to the checksum. */ |
59 | crc = crc32c(crc, buffer, sizeof(struct sctphdr) - sizeof(__u32)); | 64 | crc = sctp_crc32c(crc, buffer, sizeof(struct sctphdr) - sizeof(__u32)); |
60 | 65 | ||
61 | /* Skip checksum field of the header. */ | 66 | /* Skip checksum field of the header. */ |
62 | crc = crc32c(crc, zero, sizeof(__u32)); | 67 | crc = sctp_crc32c(crc, zero, sizeof(__u32)); |
63 | 68 | ||
64 | /* Calculate the rest of the CRC. */ | 69 | /* Calculate the rest of the CRC. */ |
65 | crc = crc32c(crc, &buffer[sizeof(struct sctphdr)], | 70 | crc = sctp_crc32c(crc, &buffer[sizeof(struct sctphdr)], |
66 | length - sizeof(struct sctphdr)); | 71 | length - sizeof(struct sctphdr)); |
67 | return crc; | 72 | return crc; |
68 | } | 73 | } |
69 | 74 | ||
70 | static inline __u32 sctp_update_cksum(__u8 *buffer, __u16 length, __u32 crc32) | 75 | static inline __be32 sctp_update_cksum(__u8 *buffer, __u16 length, __be32 crc32) |
71 | { | 76 | { |
72 | return crc32c(crc32, buffer, length); | 77 | return sctp_crc32c(crc32, buffer, length); |
73 | } | 78 | } |
74 | 79 | ||
75 | static inline __u32 sctp_end_cksum(__u32 crc32) | 80 | static inline __be32 sctp_end_cksum(__be32 crc32) |
76 | { | 81 | { |
77 | return ntohl(~crc32); | 82 | return ~crc32; |
78 | } | 83 | } |
diff --git a/net/ipv4/netfilter/nf_nat_proto_sctp.c b/net/ipv4/netfilter/nf_nat_proto_sctp.c index 82e4c0e286b8..65e470bc6123 100644 --- a/net/ipv4/netfilter/nf_nat_proto_sctp.c +++ b/net/ipv4/netfilter/nf_nat_proto_sctp.c | |||
@@ -36,7 +36,7 @@ sctp_manip_pkt(struct sk_buff *skb, | |||
36 | sctp_sctphdr_t *hdr; | 36 | sctp_sctphdr_t *hdr; |
37 | unsigned int hdroff = iphdroff + iph->ihl*4; | 37 | unsigned int hdroff = iphdroff + iph->ihl*4; |
38 | __be32 oldip, newip; | 38 | __be32 oldip, newip; |
39 | u32 crc32; | 39 | __be32 crc32; |
40 | 40 | ||
41 | if (!skb_make_writable(skb, hdroff + sizeof(*hdr))) | 41 | if (!skb_make_writable(skb, hdroff + sizeof(*hdr))) |
42 | return false; | 42 | return false; |
@@ -61,7 +61,7 @@ sctp_manip_pkt(struct sk_buff *skb, | |||
61 | crc32 = sctp_update_cksum((u8 *)skb->data, skb_headlen(skb), | 61 | crc32 = sctp_update_cksum((u8 *)skb->data, skb_headlen(skb), |
62 | crc32); | 62 | crc32); |
63 | crc32 = sctp_end_cksum(crc32); | 63 | crc32 = sctp_end_cksum(crc32); |
64 | hdr->checksum = htonl(crc32); | 64 | hdr->checksum = crc32; |
65 | 65 | ||
66 | return true; | 66 | return true; |
67 | } | 67 | } |
diff --git a/net/sctp/input.c b/net/sctp/input.c index 5ed93c05c23e..a49fa80b57b9 100644 --- a/net/sctp/input.c +++ b/net/sctp/input.c | |||
@@ -83,8 +83,8 @@ static inline int sctp_rcv_checksum(struct sk_buff *skb) | |||
83 | { | 83 | { |
84 | struct sk_buff *list = skb_shinfo(skb)->frag_list; | 84 | struct sk_buff *list = skb_shinfo(skb)->frag_list; |
85 | struct sctphdr *sh = sctp_hdr(skb); | 85 | struct sctphdr *sh = sctp_hdr(skb); |
86 | __u32 cmp = ntohl(sh->checksum); | 86 | __be32 cmp = sh->checksum; |
87 | __u32 val = sctp_start_cksum((__u8 *)sh, skb_headlen(skb)); | 87 | __be32 val = sctp_start_cksum((__u8 *)sh, skb_headlen(skb)); |
88 | 88 | ||
89 | for (; list; list = list->next) | 89 | for (; list; list = list->next) |
90 | val = sctp_update_cksum((__u8 *)list->data, skb_headlen(list), | 90 | val = sctp_update_cksum((__u8 *)list->data, skb_headlen(list), |
diff --git a/net/sctp/output.c b/net/sctp/output.c index 9a63a3fb9011..45684646b1db 100644 --- a/net/sctp/output.c +++ b/net/sctp/output.c | |||
@@ -365,7 +365,7 @@ int sctp_packet_transmit(struct sctp_packet *packet) | |||
365 | struct sctp_transport *tp = packet->transport; | 365 | struct sctp_transport *tp = packet->transport; |
366 | struct sctp_association *asoc = tp->asoc; | 366 | struct sctp_association *asoc = tp->asoc; |
367 | struct sctphdr *sh; | 367 | struct sctphdr *sh; |
368 | __u32 crc32 = 0; | 368 | __be32 crc32 = __constant_cpu_to_be32(0); |
369 | struct sk_buff *nskb; | 369 | struct sk_buff *nskb; |
370 | struct sctp_chunk *chunk, *tmp; | 370 | struct sctp_chunk *chunk, *tmp; |
371 | struct sock *sk; | 371 | struct sock *sk; |
@@ -538,7 +538,7 @@ int sctp_packet_transmit(struct sctp_packet *packet) | |||
538 | /* 3) Put the resultant value into the checksum field in the | 538 | /* 3) Put the resultant value into the checksum field in the |
539 | * common header, and leave the rest of the bits unchanged. | 539 | * common header, and leave the rest of the bits unchanged. |
540 | */ | 540 | */ |
541 | sh->checksum = htonl(crc32); | 541 | sh->checksum = crc32; |
542 | 542 | ||
543 | /* IP layer ECN support | 543 | /* IP layer ECN support |
544 | * From RFC 2481 | 544 | * From RFC 2481 |