[ovs-git] [openvswitch/ovs] 6eee2d: dp-packet: Add 'dp_packet_l3_size()'.

Aaron Conole noreply at github.com
Thu Feb 14 19:34:32 UTC 2019


  Branch: refs/heads/branch-2.8
  Home:   https://github.com/openvswitch/ovs
  Commit: 6eee2dc11e70d378dfaad9f28a57522f06017af2
      https://github.com/openvswitch/ovs/commit/6eee2dc11e70d378dfaad9f28a57522f06017af2
  Author: Darrell Ball <dlu998 at gmail.com>
  Date:   2019-02-14 (Thu, 14 Feb 2019)

  Changed paths:
    M lib/dp-packet.h

  Log Message:
  -----------
  dp-packet: Add 'dp_packet_l3_size()'.

The new api will be used in a subsequent patch.

Signed-off-by: Darrell Ball <dlu998 at gmail.com>
Signed-off-by: Ben Pfaff <blp at ovn.org>


  Commit: 3f096f73ca3e4af7d9bf2981a9db3a9850bc3232
      https://github.com/openvswitch/ovs/commit/3f096f73ca3e4af7d9bf2981a9db3a9850bc3232
  Author: Darrell Ball <dlu998 at gmail.com>
  Date:   2019-02-14 (Thu, 14 Feb 2019)

  Changed paths:
    M lib/conntrack.c

  Log Message:
  -----------
  conntrack: Exclude l2 padding in 'conn_key_extract()'.

'conn_key_extract()' in userspace conntrack is including L2
(Ethernet) pad bytes for both L3 and L4 sizes. One problem is
any packet with non-zero L2 padding can incorrectly fail L4
checksum validation.

This patch fixes conn_key_extract() by ignoring L2 pad bytes.

Fixes: a489b16854b5 ("conntrack: New userspace connection tracker.")
CC: Daniele Di Proietto <diproiettod at ovn.org>
Co-authored-by: Vishal Deep Ajmera <vishal.deep.ajmera at ericsson.com>
Co-authored-by: Venkatesan Pradeep <venkatesan.pradeep at ericsson.com>
Co-authored-by: Nitin Katiyar <nitin.katiyar at ericsson.com>
Signed-off-by: Vishal Deep Ajmera <vishal.deep.ajmera at ericsson.com>
Signed-off-by: Venkatesan Pradeep <venkatesan.pradeep at ericsson.com>
Signed-off-by: Nitin Katiyar <nitin.katiyar at ericsson.com>
Signed-off-by: Darrell Ball <dlu998 at gmail.com>
Signed-off-by: Ben Pfaff <blp at ovn.org>


  Commit: d59b7bf6b90290b0e544e3b4fb834c39651ec8a2
      https://github.com/openvswitch/ovs/commit/d59b7bf6b90290b0e544e3b4fb834c39651ec8a2
  Author: Aaron Conole <aconole at redhat.com>
  Date:   2019-02-14 (Thu, 14 Feb 2019)

  Changed paths:
    M lib/lldp/lldpd.c

  Log Message:
  -----------
  lldp: fix string warnings

lib/lldp/lldpd.c: In function :
lib/lldp/lldpd.c:520:17: warning:  output truncated before terminating nul copying as many bytes from a string as its length [-Wstringop-truncation]
                strncat(buffer, cfg->g_protocols[i].name,
                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                strlen(cfg->g_protocols[i].name));
                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

lib/lldp/lldpd.c: In function :
lib/lldp/lldpd.c:519:17: warning:  specified bound 2 equals source length [-Wstringop-overflow=]
                strncat(buffer, ", ", 2);
                ^~~~~~~~~~~~~~~~~~~~~~~~

Closer inspection shows that buffer is only used to output protocol names
when debug logging is enabled, so restructure the code a bit as well.

Signed-off-by: Aaron Conole <aconole at redhat.com>
Reviewed-by: Yifeng Sun <pkusunyifeng at gmail.com>
Signed-off-by: Ben Pfaff <blp at ovn.org>


Compare: https://github.com/openvswitch/ovs/compare/c4cc16bd4b31...d59b7bf6b902


More information about the git mailing list