diff options
author | Denis Vlasenko <vda.linux@googlemail.com> | 2007-03-08 16:14:46 +0000 |
---|---|---|
committer | Denis Vlasenko <vda.linux@googlemail.com> | 2007-03-08 16:14:46 +0000 |
commit | 8a91081d9d3bdef1d46a0629ac736b9051e32cf3 (patch) | |
tree | ec8696c1a863b042d9798c6bb6104d293a7794b6 /coreutils | |
parent | cd27c42552b57a517a7abbe836eea0c52a896630 (diff) | |
download | busybox-8a91081d9d3bdef1d46a0629ac736b9051e32cf3.tar.gz |
chown: fix comment, fix verbose message
Diffstat (limited to 'coreutils')
-rw-r--r-- | coreutils/chown.c | 31 |
1 files changed, 15 insertions, 16 deletions
diff --git a/coreutils/chown.c b/coreutils/chown.c index 439b62ac8..15d2faeae 100644 --- a/coreutils/chown.c +++ b/coreutils/chown.c @@ -7,7 +7,7 @@ * 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 */ @@ -19,9 +19,6 @@ 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 +32,24 @@ 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)) static int fileAction(const char *fileName, struct stat *statbuf, void ATTRIBUTE_UNUSED *junk, 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_func(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; } @@ -70,8 +69,8 @@ int chown_main(int argc, char **argv) /* This matches coreutils behavior (almost - see below) */ 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 +82,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) @@ -102,7 +101,7 @@ int chown_main(int argc, char **argv) retval = EXIT_FAILURE; } - if (OPT_TRAVERSETOP) + if (OPT_TRAVERSE_TOP) free(arg); } while (*++argv); |