diff options
| author |   Baptiste Daroussin <bapt@FreeBSD.org> | 2015-06-07 11:35:34 +0000 |
|---|---|---|
| committer | Baptiste Daroussin <bapt@FreeBSD.org> | 2015-06-07 11:35:34 +0000 |
| commit | 704e1b49938e3613cff8e9ac9251bcfc55aca739 (patch) | |
| tree | 3f4a882511c0cad43b1bf286163573101275ef6e /pw/pw_user.c | |
| parent | 4f3e1cb757b5ba278a748d7420bec923a9224dfc (diff) | |
| download | pw-darwin-704e1b49938e3613cff8e9ac9251bcfc55aca739.tar.gz pw-darwin-704e1b49938e3613cff8e9ac9251bcfc55aca739.tar.zst pw-darwin-704e1b49938e3613cff8e9ac9251bcfc55aca739.zip | |
Improve readability by reducing indentations levels
Diffstat (limited to 'pw/pw_user.c')
| -rw-r--r-- | pw/pw_user.c | 20 |
1 files changed, 11 insertions, 9 deletions
diff --git a/pw/pw_user.c b/pw/pw_user.c index 8a07a0e..dc33272 100644 --- a/pw/pw_user.c +++ b/pw/pw_user.c @@ -1122,15 +1122,17 @@ delete_user(struct userconf *cnf, struct passwd *pwd, struct carg *a_name, while ((grp = GETGRENT()) != NULL) { int i, j; char group[MAXLOGNAME]; - if (grp->gr_mem != NULL) { - for (i = 0; grp->gr_mem[i] != NULL; i++) { - if (!strcmp(grp->gr_mem[i], a_name->val)) { - 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); - } - } + if (grp->gr_mem == NULL) + continue; + + for (i = 0; grp->gr_mem[i] != NULL; i++) { + if (strcmp(grp->gr_mem[i], a_name->val)) + 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(); |