[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