diff options
author | Ryan Moats <rmoats@us.ibm.com> | 2016-07-24 18:36:35 +0000 |
---|---|---|
committer | Ben Pfaff <blp@ovn.org> | 2016-07-24 11:47:28 -0700 |
commit | ece9c2947dd5956708bcdb44922a2481da6b2f50 (patch) | |
tree | 5261f1d4f580784cfb9806d62b4cd0560443444f /lib | |
parent | 218f9539f9ac359c7321c1cab9432c39648285f1 (diff) | |
download | openvswitch-ece9c2947dd5956708bcdb44922a2481da6b2f50.tar.gz |
Explain initialization when using csum()
The checksum method csum() requires its output location to be
intialized to zero when that output location is part of the
checksum. Add comments to the various places where csum is
called documenting where the initialization has occurred.
Signed-off-by: Ryan Moats <rmoats@us.ibm.com>
Signed-off-by: Ben Pfaff <blp@ovn.org>
Diffstat (limited to 'lib')
-rw-r--r-- | lib/bfd.c | 1 | ||||
-rw-r--r-- | lib/flow.c | 3 | ||||
-rw-r--r-- | lib/netdev-native-tnl.c | 1 |
3 files changed, 5 insertions, 0 deletions
@@ -629,6 +629,7 @@ bfd_put_packet(struct bfd *bfd, struct dp_packet *p, ip->ip_proto = IPPROTO_UDP; put_16aligned_be32(&ip->ip_src, bfd->ip_src); put_16aligned_be32(&ip->ip_dst, bfd->ip_dst); + /* Checksum has already been zeroed by put_zeros call. */ ip->ip_csum = csum(ip, sizeof *ip); udp = dp_packet_put_zeros(p, sizeof *udp); diff --git a/lib/flow.c b/lib/flow.c index 1fa3bf459..51f68190f 100644 --- a/lib/flow.c +++ b/lib/flow.c @@ -2226,6 +2226,7 @@ flow_compose_l4(struct dp_packet *p, const struct flow *flow) icmp = dp_packet_put_zeros(p, l4_len); icmp->icmp_type = ntohs(flow->tp_src); icmp->icmp_code = ntohs(flow->tp_dst); + /* Checksum has already been zeroed by put_zeros call. */ icmp->icmp_csum = csum(icmp, ICMP_HEADER_LEN); } else if (flow->nw_proto == IPPROTO_IGMP) { struct igmp_header *igmp; @@ -2235,6 +2236,7 @@ flow_compose_l4(struct dp_packet *p, const struct flow *flow) igmp->igmp_type = ntohs(flow->tp_src); igmp->igmp_code = ntohs(flow->tp_dst); put_16aligned_be32(&igmp->group, flow->igmp_group_ip4); + /* Checksum has already been zeroed by put_zeros call. */ igmp->igmp_csum = csum(igmp, IGMP_HEADER_LEN); } else if (flow->nw_proto == IPPROTO_ICMPV6) { struct icmp6_hdr *icmp; @@ -2323,6 +2325,7 @@ flow_compose(struct dp_packet *p, const struct flow *flow) ip = dp_packet_l3(p); ip->ip_tot_len = htons(p->l4_ofs - p->l3_ofs + l4_len); + /* Checksum has already been zeroed by put_zeros call. */ ip->ip_csum = csum(ip, sizeof *ip); } else if (flow->dl_type == htons(ETH_TYPE_IPV6)) { struct ovs_16aligned_ip6_hdr *nh; diff --git a/lib/netdev-native-tnl.c b/lib/netdev-native-tnl.c index 15975ade9..ce2582f13 100644 --- a/lib/netdev-native-tnl.c +++ b/lib/netdev-native-tnl.c @@ -273,6 +273,7 @@ netdev_tnl_ip_build_header(struct ovs_action_push_tnl *data, ip->ip_frag_off = (params->flow->tunnel.flags & FLOW_TNL_F_DONT_FRAGMENT) ? htons(IP_DF) : 0; + /* Checksum has already been zeroed by eth_build_header. */ ip->ip_csum = csum(ip, sizeof *ip); data->header_len += IP_HEADER_LEN; |