d/freebsd-dev
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix a regression in pw usermod -G list

Browse Source 
The user was perperly adding the to different groups from "list" but was not
removed from the other groups it could have belong to.
While here add a regression test about this bug

PR:		185666
Reported by:	sub.mesa@gmail.com
MFC after:	1 week
This commit is contained in:
Baptiste Daroussin 2014-10-28 14:19:17 +00:00
parent ffcaf4d24f
commit 6697d7663b
Notes: svn2git 2020-12-20 02:59:44 +00:00 
svn path=/head/; revision=273779
2 changed files with 40 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
usr.sbin/pw/pw_user.c
View File

@ -751,7 +751,25 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
*/
if (mode == M_ADD || getarg(args, 'G') != NULL) {
int i;
int i, j;
/* First remove the user from all group */
SETGRENT();
while ((grp = GETGRENT()) != NULL) {
char group[MAXLOGNAME];
if (grp->gr_mem == NULL)
continue;
for (i = 0; grp->gr_mem[i] != NULL; i++) {
if (strcmp(grp->gr_mem[i] , pwd->pw_name) != 0)
continue;
for (j = i; grp->gr_mem[j] != NULL ; j++)
grp->gr_mem[j] = grp->gr_mem[j+1];
strlcpy(group, grp->gr_name, MAXLOGNAME);
chggrent(group, grp);
}
}
ENDGRENT();
/* now add to group where needed */
for (i = 0; cnf->groups[i] != NULL; i++) {
grp = GETGRNAM(cnf->groups[i]);
grp = gr_add(grp, pwd->pw_name);

21
usr.sbin/pw/tests/pw_modify.sh
View File

@ -38,8 +38,29 @@ groupmod_bug_193704_body() {
atf_check -s exit:65 -e match:"^pw: unknown group" -x pw -V ${HOME} groupshow test
}
atf_test_case usermod_bug_185666
usermod_bug_185666_head() {
atf_set "descr" "Regression test for the #185666 bug"
}
usermod_bug_185666_body() {
populate_etc_skel
atf_check -s exit:0 -x pw -V ${HOME} useradd testuser
atf_check -s exit:0 -x pw -V ${HOME} groupadd testgroup
atf_check -s exit:0 -x pw -V ${HOME} groupadd testgroup2
atf_check -s exit:0 -x pw -V ${HOME} usermod testuser -G testgroup
atf_check -o inline:"testuser:*:1001:\n" -x pw -V${HOME} groupshow testuser
atf_check -o inline:"testgroup:*:1002:testuser\n" -x pw -V ${HOME} groupshow testgroup
atf_check -o inline:"testgroup2:*:1003:\n" -x pw -V${HOME} groupshow testgroup2
atf_check -s exit:0 -x pw -V ${HOME} usermod testuser -G testgroup2
atf_check -o inline:"testuser:*:1001:\n" -x pw -V ${HOME} groupshow testuser
atf_check -o inline:"testgroup:*:1002:\n" -x pw -V ${HOME} groupshow testgroup
atf_check -o inline:"testgroup2:*:1003:testuser\n" -x pw -V ${HOME} groupshow testgroup2
}
atf_init_test_cases() {
atf_add_test_case groupmod_user
atf_add_test_case groupmod_invalid_user
atf_add_test_case groupmod_bug_193704
atf_add_test_case usermod_bug_185666
}