]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
ethtool: strset: fix header attribute index in ethnl_req_get_phydev()
authorJakub Kicinski <kuba@kernel.org>
Tue, 26 May 2026 15:35:31 +0000 (08:35 -0700)
committerJakub Kicinski <kuba@kernel.org>
Thu, 28 May 2026 00:42:09 +0000 (17:42 -0700)
strset_prepare_data() passes ETHTOOL_A_HEADER_FLAGS (3) as the header
attribute to ethnl_req_get_phydev(). This is incorrect, in the main
attr space 3 is ETHTOOL_A_STRSET_COUNTS_ONLY, not the request
header attr. The correct constant is ETHTOOL_A_STRSET_HEADER (1).

ethnl_req_get_phydev() only uses this value for the extack,
so this is not a "functionally visible"(?) bug.

Fixes: e96c93aa4be9 ("net: ethtool: strset: Allow querying phy stats by index")
Reviewed-by: Maxime Chevallier <maxime.chevallier@bootlin.com>
Link: https://patch.msgid.link/20260526153533.2779187-9-kuba@kernel.org
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
net/ethtool/strset.c

index bb1e829ba099b496929a75bba9a231142198c665..94c4718d31ae0ece0bc07f0f09728d37cb931ed1 100644 (file)
@@ -311,7 +311,7 @@ static int strset_prepare_data(const struct ethnl_req_info *req_base,
                return 0;
        }
 
-       phydev = ethnl_req_get_phydev(req_base, tb, ETHTOOL_A_HEADER_FLAGS,
+       phydev = ethnl_req_get_phydev(req_base, tb, ETHTOOL_A_STRSET_HEADER,
                                      info->extack);
 
        /* phydev can be NULL, check for errors only */