diff options
| author | Shmulik Ladkani | 2019-04-29 15:39:30 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman | 2019-05-05 14:42:36 +0200 |
| commit | 3f611a479962ed4ab3682cbb0deb476f90d513f5 (patch) | |
| tree | 48ff03a4f9380afd9754ad90aa9a375472d90e32 /net/ipv4 | |
| parent | Linux 4.19.39 (diff) | |
| download | kernel-qcow2-linux-3f611a479962ed4ab3682cbb0deb476f90d513f5.tar.gz kernel-qcow2-linux-3f611a479962ed4ab3682cbb0deb476f90d513f5.tar.xz kernel-qcow2-linux-3f611a479962ed4ab3682cbb0deb476f90d513f5.zip | |
ipv4: ip_do_fragment: Preserve skb_iif during fragmentation
[ Upstream commit d2f0c961148f65bc73eda72b9fa3a4e80973cb49 ]
Previously, during fragmentation after forwarding, skb->skb_iif isn't
preserved, i.e. 'ip_copy_metadata' does not copy skb_iif from given
'from' skb.
As a result, ip_do_fragment's creates fragments with zero skb_iif,
leading to inconsistent behavior.
Assume for example an eBPF program attached at tc egress (post
forwarding) that examines __sk_buff->ingress_ifindex:
- the correct iif is observed if forwarding path does not involve
fragmentation/refragmentation
- a bogus iif is observed if forwarding path involves
fragmentation/refragmentatiom
Fix, by preserving skb_iif during 'ip_copy_metadata'.
Signed-off-by: Shmulik Ladkani <shmulik.ladkani@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'net/ipv4')
| -rw-r--r-- | net/ipv4/ip_output.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/net/ipv4/ip_output.c b/net/ipv4/ip_output.c index 9c4e72e9c60a..73894ed12a70 100644 --- a/net/ipv4/ip_output.c +++ b/net/ipv4/ip_output.c @@ -519,6 +519,7 @@ static void ip_copy_metadata(struct sk_buff *to, struct sk_buff *from) to->pkt_type = from->pkt_type; to->priority = from->priority; to->protocol = from->protocol; + to->skb_iif = from->skb_iif; skb_dst_drop(to); skb_dst_copy(to, from); to->dev = from->dev; |