[ovs-git] [openvswitch/ovs] 98f539: datapath-windows: Do not modify port field for ICM...

GitHub noreply at github.com
Thu Aug 17 19:59:48 UTC 2017


  Branch: refs/heads/branch-2.8
  Home:   https://github.com/openvswitch/ovs
  Commit: 98f5393922979644448e1644acee8dcdf053fe10
      https://github.com/openvswitch/ovs/commit/98f5393922979644448e1644acee8dcdf053fe10
  Author: Anand Kumar <kumaranand at vmware.com>
  Date:   2017-08-17 (Thu, 17 Aug 2017)

  Changed paths:
    M datapath-windows/ovsext/Conntrack-nat.c

  Log Message:
  -----------
  datapath-windows: Do not modify port field for ICMP during SNAT/DNAT

During SNAT/DNAT, we should not be updating the port field of ct_endpoint
struct, as ICMP packets do not have port information. Since port and
icmp_id are overlapped in ct_endpoint struct, icmp_id gets changed.
As a result, NAT look up fails to find a matching entry.

This patch addresses this issue by not modifying icmp_id field during
SNAT/DNAT only for ICMP traffic

The current NAT module doesn't take the ICMP type/code into account
during the lookups. Fix this to make it similar with the other conntrack
module.

Acked-by: Alin Gabriel Serdean <aserdean at ovn.org>
Acked-by: Shashank Ram <rams at vmware.com>
Signed-off-by: Anand Kumar <kumaranand at vmware.com>
Signed-off-by: Alin Gabriel Serdean <aserdean at ovn.org>


  Commit: ea299ffb5faa3dc95e8ae7d1e85b38b6e6dd4e03
      https://github.com/openvswitch/ovs/commit/ea299ffb5faa3dc95e8ae7d1e85b38b6e6dd4e03
  Author: Anand Kumar <kumaranand at vmware.com>
  Date:   2017-08-17 (Thu, 17 Aug 2017)

  Changed paths:
    M datapath-windows/ovsext/Conntrack.c
    M datapath-windows/ovsext/Conntrack.h

  Log Message:
  -----------
  datapath-windows: Update Orig Tuple to use ICMP Type and Code

- Also add some padding for the ct_endpoint's union, so that each member
of ct_endpoint's union are of same size.

Co-authored-by: Sairam Venugopal <vsairam at vmware.com>
Signed-off-by: Anand Kumar <kumaranand at vmware.com>
Acked-by: Shashank Ram <rams at vmware.com>
Acked-by: Alin Gabriel Serdean <aserdean at ovn.org>
Signed-off-by: Alin Gabriel Serdean <aserdean at ovn.org>


Compare: https://github.com/openvswitch/ovs/compare/bed324c81358...ea299ffb5faa


More information about the git mailing list