Re: [PATCH] flow_dissector: Fix vlan header offset in __skb_flow_dissect

From: David Miller
Date: Sat Jun 22 2019 - 19:19:58 EST


From: YueHaibing <yuehaibing@xxxxxxxxxx>
Date: Thu, 20 Jun 2019 00:01:32 +0800

> @@ -785,6 +785,9 @@ bool __skb_flow_dissect(const struct sk_buff *skb,
> skb && skb_vlan_tag_present(skb)) {
> proto = skb->protocol;
> } else {
> + if (dissector_vlan == FLOW_DISSECTOR_KEY_MAX)
> + nhoff -= sizeof(*vlan);

Even if this would have turned out to be the desired fix, you would need
to get rid of the extra spaces in that last statement.