As Larry pointed out, this the off by one canbe fixed much more elegantly.

git-svn-id: svn://busybox.net/trunk/busybox@2449 69ca8d6d-28ef-0310-b511-8ec308f3f277

3 files changed, 3 insertions, 3 deletions

diff --git a/coreutils/chgrp.c b/coreutils/chgrp.c
index 83bb19463..55e22f528 100644
--- a/coreutils/chgrp.c
+++ b/coreutils/chgrp.c
@@ -71,7 +71,7 @@ int chgrp_main(int argc, char **argv)
         }
 
         /* Ok, ready to do the deed now */
-        while (optind++ < argc-1) {
+        while (++optind < argc) {
                 if (recursive_action (argv[optind], recursiveFlag, FALSE, FALSE, 
                                         fileAction, fileAction, NULL) == FALSE) {
                         return EXIT_FAILURE;
diff --git a/coreutils/chmod.c b/coreutils/chmod.c
index f22e5d06f..b1d75129b 100644
--- a/coreutils/chmod.c
+++ b/coreutils/chmod.c
@@ -66,7 +66,7 @@ int chmod_main(int argc, char **argv)
         }
 
         /* Ok, ready to do the deed now */
-        while (optind++ < argc-1) {
+        while (++optind < argc) {
                 if (recursive_action (argv[optind], recursiveFlag, FALSE, FALSE, 
                                         fileAction, fileAction, NULL) == FALSE) {
                         return EXIT_FAILURE;
diff --git a/coreutils/chown.c b/coreutils/chown.c
index 43d62b1e0..e714b2d7a 100644
--- a/coreutils/chown.c
+++ b/coreutils/chown.c
@@ -85,7 +85,7 @@ int chown_main(int argc, char **argv)
         }
 
         /* Ok, ready to do the deed now */
-        while (optind++ < argc-1) {
+        while (++optind < argc) {
                 if (recursive_action (argv[optind], recursiveFlag, FALSE, FALSE, 
                                         fileAction, fileAction, NULL) == FALSE) {
                         return EXIT_FAILURE;