Thread (2 messages) flat view 2 messages, 2 authors, 2018-01-10

Re: [PATCH net v2] ipv6: sr: fix TLVs not being copied using setsockopt

From: David Miller <davem@davemloft.net>
Date: 2018-01-10 21:04:37

From: Mathieu Xhonneux <redacted>
Date: Wed, 10 Jan 2018 13:35:49 +0000
Function ipv6_push_rthdr4 allows to add an IPv6 Segment Routing Header
to a socket through setsockopt, but the current implementation doesn't
copy possible TLVs at the end of the SRH received from userspace.

Therefore, the execution of the following branch if (sr_has_hmac(sr_phdr))
{ ... } will never complete since the len and type fields of a possible
HMAC TLV are not copied, hence seg6_get_tlv_hmac will return an error,
and the HMAC will not be computed.

This commit adds a memcpy in case TLVs have been appended to the SRH.

Fixes: a149e7c7ce812561f0fdc7a86ddc42f294e5eb3e ("ipv6: sr: add
support for SRH injection through setsockopt")
Please do not break up long Fixes: lines.  Also, you should only
provide 12 digits of the SHA1 ID.
Acked-by: David Lebrun <redacted>
Signed-off-by: Mathieu Xhonneux <redacted>
Applied, thanks.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help