Revert "geneve: make sure to pull inner header in geneve_rx()"
This reverts commit c713790069
which is
commit 1ca1ba465e55b9460e4e75dec9fff31e708fec74 upstream.
It breaks the Android kernel abi and can be brought back in the future
in an abi-safe way if it is really needed.
Bug: 161946584
Change-Id: I64aa85a2666f61fb1589d8ed8531750bac1e43fc
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
This commit is contained in:
parent
578a3af78b
commit
bbcfe35f4e
@ -218,7 +218,7 @@ static void geneve_rx(struct geneve_dev *geneve, struct geneve_sock *gs,
|
|||||||
struct genevehdr *gnvh = geneve_hdr(skb);
|
struct genevehdr *gnvh = geneve_hdr(skb);
|
||||||
struct metadata_dst *tun_dst = NULL;
|
struct metadata_dst *tun_dst = NULL;
|
||||||
unsigned int len;
|
unsigned int len;
|
||||||
int nh, err = 0;
|
int err = 0;
|
||||||
void *oiph;
|
void *oiph;
|
||||||
|
|
||||||
if (ip_tunnel_collect_metadata() || gs->collect_md) {
|
if (ip_tunnel_collect_metadata() || gs->collect_md) {
|
||||||
@ -262,23 +262,9 @@ static void geneve_rx(struct geneve_dev *geneve, struct geneve_sock *gs,
|
|||||||
goto drop;
|
goto drop;
|
||||||
}
|
}
|
||||||
|
|
||||||
/* Save offset of outer header relative to skb->head,
|
oiph = skb_network_header(skb);
|
||||||
* because we are going to reset the network header to the inner header
|
|
||||||
* and might change skb->head.
|
|
||||||
*/
|
|
||||||
nh = skb_network_header(skb) - skb->head;
|
|
||||||
|
|
||||||
skb_reset_network_header(skb);
|
skb_reset_network_header(skb);
|
||||||
|
|
||||||
if (!pskb_inet_may_pull(skb)) {
|
|
||||||
DEV_STATS_INC(geneve->dev, rx_length_errors);
|
|
||||||
DEV_STATS_INC(geneve->dev, rx_errors);
|
|
||||||
goto drop;
|
|
||||||
}
|
|
||||||
|
|
||||||
/* Get the outer header. */
|
|
||||||
oiph = skb->head + nh;
|
|
||||||
|
|
||||||
if (geneve_get_sk_family(gs) == AF_INET)
|
if (geneve_get_sk_family(gs) == AF_INET)
|
||||||
err = IP_ECN_decapsulate(oiph, skb);
|
err = IP_ECN_decapsulate(oiph, skb);
|
||||||
#if IS_ENABLED(CONFIG_IPV6)
|
#if IS_ENABLED(CONFIG_IPV6)
|
||||||
|
Loading…
Reference in New Issue
Block a user