*: remove some uses of argc

function                                             old     new   delta
whoami_main                                           34      37      +3
logname_main                                          60      63      +3
hostid_main                                           35      38      +3
ttysize_main                                         136     135      -1
nmeter_main                                          673     672      -1
logger_main                                          387     386      -1
uuencode_main                                        330     328      -2
ifupdown_main                                       2125    2123      -2
mesg_main                                            158     155      -3
free_main                                            333     330      -3
cal_main                                             902     899      -3
acpid_main                                           443     440      -3
ar_main                                              196     189      -7
find_main                                            476     467      -9
ifconfig_main                                       1235    1221     -14
------------------------------------------------------------------------------
(add/remove: 0/0 grow/shrink: 3/12 up/down: 9/-49)            Total: -40 bytes

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>

3 files changed, 14 insertions, 17 deletions

diff --git a/selinux/load_policy.c b/selinux/load_policy.c
index 4bc873e81..ea7c9132a 100644
--- a/selinux/load_policy.c
+++ b/selinux/load_policy.c
@@ -7,11 +7,11 @@
 #include "libbb.h"
 
 int load_policy_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE;
-int load_policy_main(int argc, char **argv UNUSED_PARAM)
+int load_policy_main(int argc UNUSED_PARAM, char **argv UNUSED_PARAM)
 {
         int rc;
 
-        if (argc != 1) {
+        if (argv[1]) {
                 bb_show_usage();
         }
 
diff --git a/selinux/setenforce.c b/selinux/setenforce.c
index a2d04288b..45f82238c 100644
--- a/selinux/setenforce.c
+++ b/selinux/setenforce.c
@@ -21,11 +21,11 @@ static const char *const setenforce_cmd[] = {
 };
 
 int setenforce_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE;
-int setenforce_main(int argc, char **argv)
+int setenforce_main(int argc UNUSED_PARAM, char **argv)
 {
         int i, rc;
 
-        if (argc != 2)
+        if (!argv[1] || argv[2])
                 bb_show_usage();
 
         selinux_or_die();
diff --git a/selinux/setfiles.c b/selinux/setfiles.c
index 0a4643784..4686d8042 100644
--- a/selinux/setfiles.c
+++ b/selinux/setfiles.c
@@ -490,7 +490,7 @@ static int process_one(char *name)
 }
 
 int setfiles_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE;
-int setfiles_main(int argc, char **argv)
+int setfiles_main(int argc UNUSED_PARAM, char **argv)
 {
         struct stat sb;
         int rc, i = 0;
@@ -549,6 +549,7 @@ int setfiles_main(int argc, char **argv)
                                  IF_FEATURE_SETFILES_CHECK_OPTION(&policyfile,)
                         &verbose);
         }
+        argv += optind;
 
 #if ENABLE_FEATURE_SETFILES_CHECK_OPTION
         if ((applet_name[0] == 's') && (flags & OPT_c)) {
@@ -595,24 +596,20 @@ int setfiles_main(int argc, char **argv)
                    we can support either checking against the active policy or
                    checking against a binary policy file. */
                 set_matchpathcon_canoncon(&canoncon);
-                if (argc == 1)
+                if (!argv[0])
                         bb_show_usage();
-                if (stat(argv[optind], &sb) < 0) {
-                        bb_simple_perror_msg_and_die(argv[optind]);
-                }
+                xstat(argv[0], &sb);
                 if (!S_ISREG(sb.st_mode)) {
-                        bb_error_msg_and_die("spec file %s is not a regular file", argv[optind]);
+                        bb_error_msg_and_die("spec file %s is not a regular file", argv[0]);
                 }
                 /* Load the file contexts configuration and check it. */
-                rc = matchpathcon_init(argv[optind]);
+                rc = matchpathcon_init(argv[0]);
                 if (rc < 0) {
-                        bb_simple_perror_msg_and_die(argv[optind]);
+                        bb_simple_perror_msg_and_die(argv[0]);
                 }
-
-                optind++;
-
                 if (nerr)
                         exit(EXIT_FAILURE);
+                argv++;
         }
 
         if (input_filename) {
@@ -628,9 +625,9 @@ int setfiles_main(int argc, char **argv)
                 if (ENABLE_FEATURE_CLEAN_UP)
                         fclose_if_not_stdin(f);
         } else {
-                if (optind >= argc)
+                if (!argv[0])
                         bb_show_usage();
-                for (i = optind; i < argc; i++) {
+                for (i = 0; argv[i]; i++) {
                         errors |= process_one(argv[i]);
                 }
         }