ACK: [SRU][Xenial][PATCH 1/1] Revert "sit: reload iphdr in ipip6_rcv"

Kleber Souza kleber.souza at canonical.com
Mon Jun 4 23:05:39 UTC 2018


On 05/25/18 04:48, Joseph Salisbury wrote:
> From: "David S. Miller" <davem at davemloft.net>
> 
> BugLink: http://bugs.launchpad.net/bugs/1772775
> 
> This reverts commit b699d0035836f6712917a41e7ae58d84359b8ff9.
> 
> As per Eric Dumazet, the pskb_may_pull() is a NOP in this
> particular case, so the 'iph' reload is unnecessary.
> 
> Signed-off-by: David S. Miller <davem at davemloft.net>
> (cherry picked from commit f4eb17e1efe538d4da7d574bedb00a8dafcc26b7)
> Signed-off-by: Joseph Salisbury <joseph.salisbury at canonical.com>

Acked-by: Kleber Sacilotto de Souza <kleber.souza at canonical.com>

> ---
>  net/ipv6/sit.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/net/ipv6/sit.c b/net/ipv6/sit.c
> index 51f7c32..e1e7bc0 100644
> --- a/net/ipv6/sit.c
> +++ b/net/ipv6/sit.c
> @@ -692,7 +692,6 @@ static int ipip6_rcv(struct sk_buff *skb)
>  
>  		if (iptunnel_pull_header(skb, 0, htons(ETH_P_IPV6)))
>  			goto out;
> -		iph = ip_hdr(skb);
>  
>  		err = IP_ECN_decapsulate(iph, skb);
>  		if (unlikely(err)) {
> 




More information about the kernel-team mailing list