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 | 9a58ddd97ed3d0fc7096b8793c87f971f4ec0009 (patch) | |
| tree | 2d286a0d52a9b94f44471f5f991b0e7b834274ad | |
| parent | 913db9fa365fd7f1c73ab492d2decfb718d3dbba (diff) | |
| download | pw-darwin-9a58ddd97ed3d0fc7096b8793c87f971f4ec0009.tar.gz pw-darwin-9a58ddd97ed3d0fc7096b8793c87f971f4ec0009.tar.zst pw-darwin-9a58ddd97ed3d0fc7096b8793c87f971f4ec0009.zip | |
Improve readability by reducing indentations levels
| -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(); |