aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJakub Kicinski <kuba@kernel.org>2023-06-08 09:23:44 -0700
committerDavid S. Miller <davem@davemloft.net>2023-06-12 08:50:48 +0100
commit52f79609c0c5b25fddb88e85f25ce08aa7e3fb42 (patch)
tree8cf8422dbe33c3ca4df25545de64329dc7b2c1bb
parent7ebe4eda4265642859507d1b3ca330d8c196cfe5 (diff)
downloadperf-tools-next-52f79609c0c5b25fddb88e85f25ce08aa7e3fb42.tar.gz
net: ethtool: correct MAX attribute value for stats
When compiling YNL generated code compiler complains about array-initializer-out-of-bounds. Turns out the MAX value for STATS_GRP uses the value for STATS. This may lead to random corruptions in user space (kernel itself doesn't use this value as it never parses stats). Fixes: f09ea6fb1272 ("ethtool: add a new command for reading standard stats") Signed-off-by: Jakub Kicinski <kuba@kernel.org> Reviewed-by: David Ahern <dsahern@kernel.org> Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r--include/uapi/linux/ethtool_netlink.h2
1 files changed, 1 insertions, 1 deletions
diff --git a/include/uapi/linux/ethtool_netlink.h b/include/uapi/linux/ethtool_netlink.h
index 1ebf8d455f0725..73e2c10dc2cc30 100644
--- a/include/uapi/linux/ethtool_netlink.h
+++ b/include/uapi/linux/ethtool_netlink.h
@@ -783,7 +783,7 @@ enum {
/* add new constants above here */
__ETHTOOL_A_STATS_GRP_CNT,
- ETHTOOL_A_STATS_GRP_MAX = (__ETHTOOL_A_STATS_CNT - 1)
+ ETHTOOL_A_STATS_GRP_MAX = (__ETHTOOL_A_STATS_GRP_CNT - 1)
};
enum {