On 2016/03/14 21:23, David Sterba wrote:
On Mon, Mar 14, 2016 at 09:12:36AM +0900, Satoru Takeuchi wrote:
--- a/cmds-property.c
+++ b/cmds-property.c
@@ -379,9 +379,7 @@ static int cmd_property_get(int argc, char **argv)
char *name = NULL;
int types = 0;
- clean_args_no_options(argc, argv, cmd_property_get_usage);
-
- if (check_argc_min(argc, 2) || check_argc_max(argc, 5))
+ if (check_argc_min(argc, 2))
usage(cmd_property_get_usage);
parse_args(argc, argv, cmd_property_get_usage, &types, &object, &name,
We still need to check the number of non-option arguments here, when the
optind is set from parse_args.
OK, I'll send a patch which checks the number after getopt.
Thanks,
Satoru
@@ -415,9 +413,7 @@ static int cmd_property_set(int argc, char **argv)
- if (check_argc_min(argc, 4) || check_argc_max(argc, 6))
+ if (check_argc_min(argc, 4))
usage(cmd_property_set_usage);
...
parse_args(argc, argv, cmd_property_set_usage, &types,
@@ -446,9 +442,7 @@ static int cmd_property_list(int argc, char **argv)
- if (check_argc_min(argc, 2) || check_argc_max(argc, 4))
+ if (check_argc_min(argc, 2))
...
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
