lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <20190221123908.7196-8-maximmi@mellanox.com> Date: Thu, 21 Feb 2019 12:40:05 +0000 From: Maxim Mikityanskiy <maximmi@...lanox.com> To: "David S. Miller" <davem@...emloft.net>, Saeed Mahameed <saeedm@...lanox.com>, Willem de Bruijn <willemb@...gle.com>, Jason Wang <jasowang@...hat.com>, Eric Dumazet <edumazet@...gle.com> CC: "netdev@...r.kernel.org" <netdev@...r.kernel.org>, Eran Ben Elisha <eranbe@...lanox.com>, Tariq Toukan <tariqt@...lanox.com>, Maxim Mikityanskiy <maximmi@...lanox.com> Subject: [PATCH net-next v2 7/7] net/mlx5e: Trust kernel regarding transport offset After AF_PACKET is fixed to calculate the transport header offset correctly, trust the value set by the kernel. If the offset wasn't set, it means there is no transport header in the packet. Signed-off-by: Maxim Mikityanskiy <maximmi@...lanox.com> Reviewed-by: Saeed Mahameed <saeedm@...lanox.com> --- drivers/net/ethernet/mellanox/mlx5/core/en_tx.c | 4 ---- 1 file changed, 4 deletions(-) diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en_tx.c b/drivers/net/ethernet/mellanox/mlx5/core/en_tx.c index 6ca834702306..e7aae45a01f8 100644 --- a/drivers/net/ethernet/mellanox/mlx5/core/en_tx.c +++ b/drivers/net/ethernet/mellanox/mlx5/core/en_tx.c @@ -148,12 +148,8 @@ static inline int mlx5e_skb_l2_header_offset(struct sk_buff *skb) static inline int mlx5e_skb_l3_header_offset(struct sk_buff *skb) { - struct flow_keys keys; - if (skb_transport_header_was_set(skb)) return skb_transport_offset(skb); - else if (skb_flow_dissect_flow_keys(skb, &keys, 0)) - return keys.control.thoff; else return mlx5e_skb_l2_header_offset(skb); } -- 2.19.1
Powered by blists - more mailing lists