netdev - Re: [PATCH net] ethtool: Fix context creation with no parameters

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] [day] [month] [year] [list]

Message-ID: <25510215-6499-4c36-89c5-ca2a2a4f4134@nvidia.com>
Date: Wed, 7 Aug 2024 19:48:14 +0300
From: Gal Pressman <gal@...dia.com>
To: "David S. Miller" <davem@...emloft.net>, Jakub Kicinski <kuba@...nel.org>
Cc: netdev@...r.kernel.org, Eric Dumazet <edumazet@...gle.com>,
 Paolo Abeni <pabeni@...hat.com>, Dragos Tatulea <dtatulea@...dia.com>,
 Jianbo Liu <jianbol@...dia.com>
Subject: Re: [PATCH net] ethtool: Fix context creation with no parameters

On 07/08/2024 19:14, Gal Pressman wrote:
> On 07/08/2024 16:25, Gal Pressman wrote:
>> diff --git a/net/ethtool/ioctl.c b/net/ethtool/ioctl.c
>> index 8ca13208d240..2fdbdcfa1506 100644
>> --- a/net/ethtool/ioctl.c
>> +++ b/net/ethtool/ioctl.c
>> @@ -1372,14 +1372,15 @@ static noinline_for_stack int ethtool_set_rxfh(struct net_device *dev,
>>  	/* If either indir, hash key or function is valid, proceed further.
> 
> This comment is wrong, the check doesn't really verify the hash function
> is valid. I'll remove it in my fix unless you have any objections.

Either this comment is horribly wrong or it's too late for me to look at
code..
The code doesn't proceed if either indir or key are valid, it returns if
either of them are invalid, I'm removing this comment altogether..