Fix off-by-one bugs in subcommand argument passing.
authorTim Potter <tpot@hp.com>
Sun, 17 Feb 2008 23:24:51 +0000 (10:24 +1100)
committerGünther Deschner <gd@samba.org>
Mon, 18 Feb 2008 09:15:53 +0000 (10:15 +0100)
(This used to be commit d85a2782a483e563a8b68924d4897a145bc25896)

source3/rpcclient/cmd_samr.c

index f8b8ba893ca20530759730333e875e36bf2fb3ce..8b4ee47a6736b31e6f82c54370a2f1aa67aec60e 100644 (file)
@@ -1044,8 +1044,8 @@ static NTSTATUS cmd_samr_enum_domains(struct rpc_pipe_client *cli,
                return NT_STATUS_OK;
        }
 
-       if (argc > 2) {
-               sscanf(argv[2], "%x", &access_mask);
+       if (argc > 1) {
+               sscanf(argv[1], "%x", &access_mask);
        }
 
        /* Get sam policy handle */
@@ -1194,11 +1194,11 @@ static NTSTATUS cmd_samr_query_aliasinfo(struct rpc_pipe_client *cli,
 
        sscanf(argv[2], "%i", &alias_rid);
 
-       if (argc > 3) {
+       if (argc > 2) {
                level = atoi(argv[3]);
        }
 
-       if (argc > 4) {
+       if (argc > 3) {
                sscanf(argv[4], "%x", &access_mask);
        }