[PATCH ethtool v2 1/2] netlink: rings: require a value for hds-thresh

[Dave Seddon <dave.seddon.ca@gmail.com>]

hds-thresh (-G) takes a value, but its parser entry sets min_argc to 0.
Given as the last word on the line, the value handler reads the NULL
that ends argv and the error message prints it:

  # ethtool -G eth0 hds-thresh
  ethtool (-G): invalid value '(null)' for parameter 'hds-thresh'

Set min_argc to 1, as every other value-taking parameter does, so the
parser reports the missing value itself:

  # ethtool -G eth0 hds-thresh
  ethtool (-G): no value for parameter 'hds-thresh'

Only the message changes: the exit status was already 1 and no request
was sent.

Fixes: 66e5dcf295c0 ("ethtool: Add support for configuring hds-thresh")
Assisted-by: Claude:claude-opus-5-5
Signed-off-by: Dave Seddon <dave.seddon.ca@gmail.com>
---
 netlink/rings.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/netlink/rings.c b/netlink/rings.c
index 5c695ab..22552df 100644
--- a/netlink/rings.c
+++ b/netlink/rings.c
@@ -202,7 +202,7 @@ static const struct param_parser sring_params[] = {
 		.arg		= "hds-thresh",
 		.type		= ETHTOOL_A_RINGS_HDS_THRESH,
 		.handler	= nl_parse_direct_u32,
-		.min_argc	= 0,
+		.min_argc	= 1,
 	},
 	{}
 };
-- 
2.56.0