- add libbb function str_tolower to convert a string to lowercase.

[oweals/busybox.git] / coreutils / chown.c

diff --git a/coreutils/chown.c b/coreutils/chown.c
index 439b62ac808f8675d7486a7c57cd01726cbec800..e64a39c3e8ce6f9dc1c8a8610cd817fbfc14ecc2 100644 (file)
--- a/coreutils/chown.c
+++ b/coreutils/chown.c
@@ -7,21 +7,14 @@
  * Licensed under GPLv2 or later, see file LICENSE in this tarball for details.
  */
 
-/* BB_AUDIT SUSv3 defects - unsupported options -H, -L, and -P. */
+/* BB_AUDIT SUSv3 defects - none? */
 /* BB_AUDIT GNU defects - unsupported long options. */
 /* http://www.opengroup.org/onlinepubs/007904975/utilities/chown.html */
 
 #include "busybox.h"
 
-static struct bb_uidgid_t ugid = { -1, -1 };
-
-static int (*chown_func)(const char *, uid_t, gid_t) = chown;
-
 #define OPT_STR     ("Rh" USE_DESKTOP("vcfLHP"))
 #define BIT_RECURSE 1
-#define BIT_NODEREF 2
-#define BIT_TRAVERSE 0x20
-#define BIT_TRAVERSETOP (0x20|0x40)
 #define OPT_RECURSE (option_mask32 & 1)
 #define OPT_NODEREF (option_mask32 & 2)
 #define OPT_VERBOSE (USE_DESKTOP(option_mask32 & 0x04) SKIP_DESKTOP(0))
@@ -35,22 +28,28 @@ static int (*chown_func)(const char *, uid_t, gid_t) = chown;
  * "Specifying more than one of -H, -L, and -P is not an error.
  * The last option specified shall determine the behavior of the utility." */
 /* -L */
-#define OPT_TRAVERSE    (USE_DESKTOP(option_mask32 & BIT_TRAVERSE) SKIP_DESKTOP(0))
+#define BIT_TRAVERSE 0x20
+#define OPT_TRAVERSE (USE_DESKTOP(option_mask32 & BIT_TRAVERSE) SKIP_DESKTOP(0))
 /* -H or -L */
-#define OPT_TRAVERSETOP (USE_DESKTOP(option_mask32 & BIT_TRAVERSETOP) SKIP_DESKTOP(0))
+#define BIT_TRAVERSE_TOP (0x20|0x40)
+#define OPT_TRAVERSE_TOP (USE_DESKTOP(option_mask32 & BIT_TRAVERSE_TOP) SKIP_DESKTOP(0))
+
+typedef int (*chown_fptr)(const char *, uid_t, gid_t);
+
+static struct bb_uidgid_t ugid = { -1, -1 };
 
 static int fileAction(const char *fileName, struct stat *statbuf,
-               void ATTRIBUTE_UNUSED *junk, int depth)
+               void *cf, int depth)
 {
-       if (!chown_func(fileName,
-                       (ugid.uid == (uid_t)-1) ? statbuf->st_uid : ugid.uid,
-                       (ugid.gid == (gid_t)-1) ? statbuf->st_gid : ugid.gid)
-       ) {
+       uid_t u = (ugid.uid == (uid_t)-1) ? statbuf->st_uid : ugid.uid;
+       gid_t g = (ugid.gid == (gid_t)-1) ? statbuf->st_gid : ugid.gid;
+
+       if (!((chown_fptr)cf)(fileName, u, g)) {
                if (OPT_VERBOSE
-                || (OPT_CHANGED && (statbuf->st_uid != ugid.uid || statbuf->st_gid != ugid.gid))
+                || (OPT_CHANGED && (statbuf->st_uid != u || statbuf->st_gid != g))
                ) {
                        printf("changed ownership of '%s' to %u:%u\n",
-                                       fileName, ugid.uid, ugid.gid);
+                                       fileName, (unsigned)u, (unsigned)g);
                }
                return TRUE;
        }
@@ -63,15 +62,17 @@ int chown_main(int argc, char **argv);
 int chown_main(int argc, char **argv)
 {
        int retval = EXIT_SUCCESS;
+       chown_fptr chown_func;
 
        opt_complementary = "-2";
        getopt32(argc, argv, OPT_STR);
        argv += optind;
 
        /* This matches coreutils behavior (almost - see below) */
+       chown_func = chown;
        if (OPT_NODEREF
-           /* || (OPT_RECURSE && !OPT_TRAVERSETOP): */
-           USE_DESKTOP( || (option_mask32 & (BIT_RECURSE|BIT_TRAVERSETOP)) == BIT_RECURSE)
+           /* || (OPT_RECURSE && !OPT_TRAVERSE_TOP): */
+           USE_DESKTOP( || (option_mask32 & (BIT_RECURSE|BIT_TRAVERSE_TOP)) == BIT_RECURSE)
        ) {
                chown_func = lchown;
        }
@@ -83,7 +84,7 @@ int chown_main(int argc, char **argv)
        do {
                char *arg = *argv;
 
-               if (OPT_TRAVERSETOP) {
+               if (OPT_TRAVERSE_TOP) {
                        /* resolves symlink (even recursive) */
                        arg = xmalloc_realpath(arg);
                        if (!arg)
@@ -91,18 +92,17 @@ int chown_main(int argc, char **argv)
                }
 
                if (!recursive_action(arg,
-                               OPT_RECURSE,    // recurse
-                               OPT_TRAVERSE,   // follow links if -L
-                               FALSE,          // depth first
-                               fileAction,     // file action
-                               fileAction,     // dir action
-                               NULL,           // user data
-                               0)              // depth
+                               (OPT_RECURSE ? ACTION_RECURSE : 0) | /* recurse */
+                               (OPT_TRAVERSE ? ACTION_FOLLOWLINKS : 0),/* follow links if -L */
+                               fileAction,     /* file action */
+                               fileAction,     /* dir action */
+                               chown_func,     /* user data */
+                               0)              /* depth */
                ) {
                        retval = EXIT_FAILURE;
                }
 
-               if (OPT_TRAVERSETOP)
+               if (OPT_TRAVERSE_TOP)
                        free(arg);
        } while (*++argv);