netdev - Re: [PATCH net,v2] ipvlan: implement .parse_protocol hook function in ipvlan_header

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

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date: Fri, 01 Dec 2023 10:22:02 -0500
From: Willem de Bruijn <willemdebruijn.kernel@...il.com>
To: shaozhengchao <shaozhengchao@...wei.com>, 
 netdev@...r.kernel.org, 
 davem@...emloft.net, 
 edumazet@...gle.com, 
 kuba@...nel.org, 
 pabeni@...hat.com
Cc: luwei32@...wei.com, 
 fw@...len.de, 
 maheshb@...gle.com, 
 weiyongjun1@...wei.com, 
 yuehaibing@...wei.com
Subject: Re: [PATCH net,v2] ipvlan: implement .parse_protocol hook function in
 ipvlan_header_ops

shaozhengchao wrote:
> 
> 
> On 2023/12/1 10:55, Zhengchao Shao wrote:
> > The .parse_protocol hook function in the ipvlan_header_ops structure is
> > not implemented. As a result, when the AF_PACKET family is used to send
> > packets, skb->protocol will be set to 0.
> > Ipvlan is a device of type ARPHRD_ETHER (ether_setup). Therefore, use
> > eth_header_parse_protocol function to obtain the protocol.
> > 
> > Fixes: 2ad7bf363841 ("ipvlan: Initial check-in of the IPVLAN driver.")
> 
> Maybe Fixes should be: 75c65772c3d1 ("net/packet: Ask driver for
> protocol if not provided by user")

Definitely not anything older than the introduction of
header_ops.parse_protocol.

I gave my +1 when it targeted net-next, so imho this is not really
stable material anyhow.