diff options
| author | Denys Vlasenko | 2011-05-21 21:32:40 +0200 |
|---|---|---|
| committer | Denys Vlasenko | 2011-05-21 21:32:40 +0200 |
| commit | 9b90d9b503c7be343ae26ce7f834b1865ab66013 (patch) | |
| tree | 0fab15423e3220d9790066422306f43b90d5338e | |
| parent | 569e930768922812ce5169a3058b65fff9196308 (diff) | |
| download | busybox-9b90d9b503c7be343ae26ce7f834b1865ab66013.zip busybox-9b90d9b503c7be343ae26ce7f834b1865ab66013.tar.gz | |
adduser: better logic for choosing addgroup -g/--gid
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
| -rw-r--r-- | loginutils/adduser.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/loginutils/adduser.c b/loginutils/adduser.c index 0e06ab2..47c674b 100644 --- a/loginutils/adduser.c +++ b/loginutils/adduser.c @@ -95,12 +95,14 @@ static void addgroup_wrapper(struct passwd *p, const char *group_name) } else { /* Add user to his own group with the first free gid * found in passwd_study. - * We try to use --gid, not -g, because "standard" addgroup - * has no such short option, it has only long --gid. */ -#if ENABLE_FEATURE_ADDGROUP_LONG_OPTIONS +#if ENABLE_FEATURE_ADDGROUP_LONG_OPTIONS || !ENABLE_ADDGROUP + /* We try to use --gid, not -g, because "standard" addgroup + * has no short option -g, it has only long --gid. + */ argv[1] = (char*)"--gid"; #else + /* Breaks if system in fact does NOT use busybox addgroup */ argv[1] = (char*)"-g"; #endif argv[2] = utoa(p->pw_gid); |