Cleanup some warnings and a really obvious bug.

[oweals/busybox.git] / chgrp.c

diff --git a/chgrp.c b/chgrp.c
index 83bb194635da11a22f64d958a7af2cb772a475c8..fbc1036a86c9ef582887d7ddf41769f3b7f7718a 100644 (file)
--- a/chgrp.c
+++ b/chgrp.c
@@ -34,7 +34,7 @@
 #endif
 
 
-static long gid = -1;
+static long gid;
 
 static int fileAction(const char *fileName, struct stat *statbuf, void* junk)
 {
@@ -56,6 +56,7 @@ int chgrp_main(int argc, char **argv)
                switch (opt) {
                        case 'R':
                                recursiveFlag = TRUE;
+                               break;
                        default:
                                show_usage();
                }
@@ -71,7 +72,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;