summaryrefslogtreecommitdiff
path: root/loginutils
diff options
context:
space:
mode:
authorDenys Vlasenko2010-01-10 04:25:05 +0100
committerDenys Vlasenko2010-01-10 04:25:05 +0100
commit38d3e64d2593aa5bc6bb578e1cd9a73a2d4b26fa (patch)
tree7215b21dee93cbbc4e2739f7cd99e11c56fbd49b /loginutils
parent1fbcb805a370e2e5f8ccfd4a45dff3ae401cd8bb (diff)
downloadbusybox-38d3e64d2593aa5bc6bb578e1cd9a73a2d4b26fa.zip
busybox-38d3e64d2593aa5bc6bb578e1cd9a73a2d4b26fa.tar.gz
adduser: create a group for net user too
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'loginutils')
-rw-r--r--loginutils/adduser.c24
1 files changed, 11 insertions, 13 deletions
diff --git a/loginutils/adduser.c b/loginutils/adduser.c
index 5f593ac..da41fd7 100644
--- a/loginutils/adduser.c
+++ b/loginutils/adduser.c
@@ -69,8 +69,11 @@ static void addgroup_wrapper(struct passwd *p, const char *group_name)
{
char *cmd;
- cmd = xasprintf("addgroup '%s' '%s'",
- p->pw_name, group_name);
+ if (group_name) /* Add user to existing group */
+ cmd = xasprintf("addgroup '%s' '%s'", p->pw_name, group_name);
+ else /* Add user to his own group with the first free gid found in passwd_study */
+ cmd = xasprintf("addgroup -g %u '%s'", (unsigned)p->pw_gid, p->pw_name);
+ /* Warning: to be compatible with external addgroup programs we should use --gid instead */
system(cmd);
free(cmd);
}
@@ -79,10 +82,8 @@ static void passwd_wrapper(const char *login) NORETURN;
static void passwd_wrapper(const char *login)
{
- static const char prog[] ALIGN1 = "passwd";
-
- BB_EXECLP(prog, prog, login, NULL);
- bb_error_msg_and_die("can't execute %s, you must set password manually", prog);
+ BB_EXECLP("passwd", "passwd", login, NULL);
+ bb_error_msg_and_die("can't execute passwd, you must set password manually");
}
#if ENABLE_FEATURE_ADDUSER_LONG_OPTIONS
@@ -189,10 +190,7 @@ int adduser_main(int argc UNUSED_PARAM, char **argv)
#endif
/* add to group */
- /* addgroup should be responsible for dealing w/ gshadow */
- /* if using a pre-existing group, don't create one */
- if (usegroup)
- addgroup_wrapper(&pw, usegroup);
+ addgroup_wrapper(&pw, usegroup);
/* clear the umask for this process so it doesn't
* screw up the permissions on the mkdir and chown. */
@@ -201,9 +199,9 @@ int adduser_main(int argc UNUSED_PARAM, char **argv)
/* set the owner and group so it is owned by the new user,
* then fix up the permissions to 2755. Can't do it before
* since chown will clear the setgid bit */
- if (mkdir(pw.pw_dir, 0755)
- || chown(pw.pw_dir, pw.pw_uid, pw.pw_gid)
- || chmod(pw.pw_dir, 02755) /* set setgid bit on homedir */
+ if ((mkdir(pw.pw_dir, 0755) != 0 && errno != EEXIST)
+ || chown(pw.pw_dir, pw.pw_uid, pw.pw_gid) != 0
+ || chmod(pw.pw_dir, 02755) != 0 /* set setgid bit on homedir */
) {
bb_simple_perror_msg(pw.pw_dir);
}