Apple-2gs-SW/hush
1
0
Fork 0
mirror of https://github.com/sheumann/hush.git synced 2024-11-14 05:05:59 +00:00
Code Issues Projects Releases Wiki Activity

adduser: use better argv passing to addgroup

Browse Source 
function                                             old     new   delta
adduser_main                                         855     882     +27

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
This commit is contained in:
Denys Vlasenko 2011-02-27 04:03:27 +01:00
parent e3b1a1fd28
commit 143c5baf6c
1 changed files with 17 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
loginutils/adduser.c
View File

@ -67,15 +67,24 @@ static void passwd_study(struct passwd *p)
static void addgroup_wrapper(struct passwd *p, const char *group_name)
{
char *cmd;
char *argv[5];
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);
argv[0] = (char*)"addgroup";
if (group_name) {
/* Add user to existing group */
argv[1] = p->pw_name;
argv[2] = (char*)group_name;
argv[3] = NULL;
} else {
/* Add user to his own group with the first free gid found in passwd_study */
//TODO: to be compatible with external addgroup programs we should use --gid instead...
argv[1] = (char*)"-g";
argv[2] = utoa(p->pw_gid);
argv[3] = p->pw_name;
argv[4] = NULL;
}
spawn_and_wait(argv);
}
static void passwd_wrapper(const char *login_name) NORETURN;