From: Johannes Berg <[email protected]>

This accidentally got wired up to the *get* policy instead
of the *set* policy, causing operations to be rejected. Fix
it by wiring up the correct policy instead.

Fixes: 5028588b62cb ("ethtool: wire up set policies to ops")
Reported-by: Leon Romanovsky <[email protected]>
Signed-off-by: Johannes Berg <[email protected]>
---
 net/ethtool/netlink.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/net/ethtool/netlink.c b/net/ethtool/netlink.c
index 8a85a4e6be9b..50d3c8896f91 100644
--- a/net/ethtool/netlink.c
+++ b/net/ethtool/netlink.c
@@ -830,8 +830,8 @@ static const struct genl_ops ethtool_genl_ops[] = {
                .cmd    = ETHTOOL_MSG_CHANNELS_SET,
                .flags  = GENL_UNS_ADMIN_PERM,
                .doit   = ethnl_set_channels,
-               .policy = ethnl_channels_get_policy,
-               .maxattr = ARRAY_SIZE(ethnl_channels_get_policy) - 1,
+               .policy = ethnl_channels_set_policy,
+               .maxattr = ARRAY_SIZE(ethnl_channels_set_policy) - 1,
        },
        {
                .cmd    = ETHTOOL_MSG_COALESCE_GET,
-- 
2.26.2

Reply via email to