diff options
| author | Denis Vlasenko | 2008-09-14 19:31:36 +0000 |
|---|---|---|
| committer | Denis Vlasenko | 2008-09-14 19:31:36 +0000 |
| commit | 0d602e9386cc651ca0ba534b309fce0582fa8e99 (patch) | |
| tree | 7b159d92dbf169f762ae991fc0fc3e771eda101b | |
| parent | 2bc5c033dc9c7a6b32535eb0e13a676684e1c17d (diff) | |
| download | busybox-0d602e9386cc651ca0ba534b309fce0582fa8e99.zip busybox-0d602e9386cc651ca0ba534b309fce0582fa8e99.tar.gz | |
id: unsigned long is excessive for option bitmask
| -rw-r--r-- | coreutils/id.c | 22 |
1 files changed, 8 insertions, 14 deletions
diff --git a/coreutils/id.c b/coreutils/id.c index cf642c2..5225f35 100644 --- a/coreutils/id.c +++ b/coreutils/id.c @@ -25,7 +25,7 @@ #define JUST_CONTEXT 32 #endif -static int printf_full(unsigned int id, const char *arg, const char *prefix) +static int printf_full(unsigned id, const char *arg, const char *prefix) { const char *fmt = "%s%u"; int status = EXIT_FAILURE; @@ -46,7 +46,7 @@ int id_main(int argc UNUSED_PARAM, char **argv) gid_t gid; gid_t *groups; int n; - unsigned long flags; + unsigned flags; short status; #if ENABLE_SELINUX security_context_t scontext; @@ -81,7 +81,7 @@ int id_main(int argc UNUSED_PARAM, char **argv) if (flags & NAME_NOT_NUMBER) printf("%s", bb_getgrgid(NULL, 0, *groups++)); else - printf("%d", (int) *groups++); + printf("%u", (unsigned) *groups++); bb_putchar((n > 0) ? ' ' : '\n'); } /* exit */ @@ -91,14 +91,14 @@ int id_main(int argc UNUSED_PARAM, char **argv) if (flags & (JUST_GROUP | JUST_USER USE_SELINUX(| JUST_CONTEXT))) { /* JUST_GROUP and JUST_USER are mutually exclusive */ if (flags & NAME_NOT_NUMBER) { - /* bb_getXXXid(-1) exit on failure, puts cannot segfault */ + /* bb_getXXXid(-1) exits on failure, puts cannot segfault */ puts((flags & JUST_USER) ? bb_getpwuid(NULL, -1, uid) : bb_getgrgid(NULL, -1, gid)); } else { if (flags & JUST_USER) { - printf("%u\n", uid); + printf("%u\n", (unsigned)uid); } if (flags & JUST_GROUP) { - printf("%u\n", gid); + printf("%u\n", (unsigned)gid); } } @@ -136,16 +136,10 @@ int id_main(int argc UNUSED_PARAM, char **argv) #if ENABLE_SELINUX if (is_selinux_enabled()) { security_context_t mysid; - const char *context; - - context = "unknown"; getcon(&mysid); - if (mysid) { - context = alloca(strlen(mysid) + 1); - strcpy((char*)context, mysid); + printf(" context=%s", mysid ? mysid : "unknown"); + if (mysid) /* TODO: maybe freecon(NULL) is harmless? */ freecon(mysid); - } - printf(" context=%s", context); } #endif |