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

author: Eric Andersen <andersen@codepoet.org> 2001-04-26 15:41:51 +0000
committer: Eric Andersen <andersen@codepoet.org> 2001-04-26 15:41:51 +0000
commit: 3c3277f0bda79b66b35e5e965e5340624d29a459 (patch)
tree: 4110e6df0e3a6fe01d34bf575cb5c0039def1ec0
parent: 30592a54514ca52253ed5f2eff64684e32d7ff05 (diff)
download: busybox-w32-3c3277f0bda79b66b35e5e965e5340624d29a459.tar.gz
busybox-w32-3c3277f0bda79b66b35e5e965e5340624d29a459.tar.bz2
busybox-w32-3c3277f0bda79b66b35e5e965e5340624d29a459.zip
6 files changed, 6 insertions, 6 deletions
diff --git a/chgrp.c b/chgrp.c
index 83bb19463..55e22f528 100644
--- a/chgrp.c
+++ b/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/chmod.c b/chmod.c
index f22e5d06f..b1d75129b 100644
--- a/chmod.c
+++ b/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/chown.c b/chown.c
index 43d62b1e0..e714b2d7a 100644
--- a/chown.c
+++ b/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;
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;
author	Eric Andersen <andersen@codepoet.org>	2001-04-26 15:41:51 +0000
committer	Eric Andersen <andersen@codepoet.org>	2001-04-26 15:41:51 +0000
commit	3c3277f0bda79b66b35e5e965e5340624d29a459 (patch)
tree	4110e6df0e3a6fe01d34bf575cb5c0039def1ec0
parent	30592a54514ca52253ed5f2eff64684e32d7ff05 (diff)
download	busybox-w32-3c3277f0bda79b66b35e5e965e5340624d29a459.tar.gz busybox-w32-3c3277f0bda79b66b35e5e965e5340624d29a459.tar.bz2 busybox-w32-3c3277f0bda79b66b35e5e965e5340624d29a459.zip

diff --git a/chgrp.c b/chgrp.c index 83bb19463..55e22f528 100644 --- a/chgrp.c +++ b/chgrp.c
@@ -71,7 +71,7 @@ int chgrp_main(int argc, char **argv)
71	}	71	}
72		72
73	/* Ok, ready to do the deed now */	73	/* Ok, ready to do the deed now */
74	while (optind++ < argc-1) {	74	while (++optind < argc) {
75	if (recursive_action (argv[optind], recursiveFlag, FALSE, FALSE,	75	if (recursive_action (argv[optind], recursiveFlag, FALSE, FALSE,
76	fileAction, fileAction, NULL) == FALSE) {	76	fileAction, fileAction, NULL) == FALSE) {
77	return EXIT_FAILURE;	77	return EXIT_FAILURE;


diff --git a/chmod.c b/chmod.c index f22e5d06f..b1d75129b 100644 --- a/chmod.c +++ b/chmod.c
@@ -66,7 +66,7 @@ int chmod_main(int argc, char **argv)
66	}	66	}
67		67
68	/* Ok, ready to do the deed now */	68	/* Ok, ready to do the deed now */
69	while (optind++ < argc-1) {	69	while (++optind < argc) {
70	if (recursive_action (argv[optind], recursiveFlag, FALSE, FALSE,	70	if (recursive_action (argv[optind], recursiveFlag, FALSE, FALSE,
71	fileAction, fileAction, NULL) == FALSE) {	71	fileAction, fileAction, NULL) == FALSE) {
72	return EXIT_FAILURE;	72	return EXIT_FAILURE;


diff --git a/chown.c b/chown.c index 43d62b1e0..e714b2d7a 100644 --- a/chown.c +++ b/chown.c
@@ -85,7 +85,7 @@ int chown_main(int argc, char **argv)
85	}	85	}
86		86
87	/* Ok, ready to do the deed now */	87	/* Ok, ready to do the deed now */
88	while (optind++ < argc-1) {	88	while (++optind < argc) {
89	if (recursive_action (argv[optind], recursiveFlag, FALSE, FALSE,	89	if (recursive_action (argv[optind], recursiveFlag, FALSE, FALSE,
90	fileAction, fileAction, NULL) == FALSE) {	90	fileAction, fileAction, NULL) == FALSE) {
91	return EXIT_FAILURE;	91	return EXIT_FAILURE;


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)
71	}	71	}
72		72
73	/* Ok, ready to do the deed now */	73	/* Ok, ready to do the deed now */
74	while (optind++ < argc-1) {	74	while (++optind < argc) {
75	if (recursive_action (argv[optind], recursiveFlag, FALSE, FALSE,	75	if (recursive_action (argv[optind], recursiveFlag, FALSE, FALSE,
76	fileAction, fileAction, NULL) == FALSE) {	76	fileAction, fileAction, NULL) == FALSE) {
77	return EXIT_FAILURE;	77	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)
66	}	66	}
67		67
68	/* Ok, ready to do the deed now */	68	/* Ok, ready to do the deed now */
69	while (optind++ < argc-1) {	69	while (++optind < argc) {
70	if (recursive_action (argv[optind], recursiveFlag, FALSE, FALSE,	70	if (recursive_action (argv[optind], recursiveFlag, FALSE, FALSE,
71	fileAction, fileAction, NULL) == FALSE) {	71	fileAction, fileAction, NULL) == FALSE) {
72	return EXIT_FAILURE;	72	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)
85	}	85	}
86		86
87	/* Ok, ready to do the deed now */	87	/* Ok, ready to do the deed now */
88	while (optind++ < argc-1) {	88	while (++optind < argc) {
89	if (recursive_action (argv[optind], recursiveFlag, FALSE, FALSE,	89	if (recursive_action (argv[optind], recursiveFlag, FALSE, FALSE,
90	fileAction, fileAction, NULL) == FALSE) {	90	fileAction, fileAction, NULL) == FALSE) {
91	return EXIT_FAILURE;	91	return EXIT_FAILURE;