diff options
author | julian <julian@FreeBSD.org> | 2014-03-06 19:26:08 +0000 |
---|---|---|
committer | julian <julian@FreeBSD.org> | 2014-03-06 19:26:08 +0000 |
commit | d8e87db4718bb459f5a3f32d67d3d8e5f75715a5 (patch) | |
tree | e7975b4a3e12d74f43ba50c20efeb2a6c458d778 | |
parent | 4f9a1ea514e7152874e5454ca0eb02fefc769e3c (diff) | |
download | FreeBSD-src-d8e87db4718bb459f5a3f32d67d3d8e5f75715a5.zip FreeBSD-src-d8e87db4718bb459f5a3f32d67d3d8e5f75715a5.tar.gz |
Stop pw(8) from segfaulting when given certain input
PR:187310
Submitted by: Kim Shrier
Obtained from: bug
MFC after: 1 week
-rw-r--r-- | usr.sbin/pw/pw_group.c | 19 |
1 files changed, 13 insertions, 6 deletions
diff --git a/usr.sbin/pw/pw_group.c b/usr.sbin/pw/pw_group.c index 3259412..391e477 100644 --- a/usr.sbin/pw/pw_group.c +++ b/usr.sbin/pw/pw_group.c @@ -227,10 +227,12 @@ pw_group(struct userconf * cnf, int mode, struct cargs * args) else if (arg->ch == 'm') { int k = 0; - while (grp->gr_mem[k] != NULL) { - if (extendarray(&members, &grmembers, i + 2) != -1) - members[i++] = grp->gr_mem[k]; - k++; + if (grp->gr_mem != NULL) { + while (grp->gr_mem[k] != NULL) { + if (extendarray(&members, &grmembers, i + 2) != -1) + members[i++] = grp->gr_mem[k]; + k++; + } } } @@ -311,6 +313,9 @@ delete_members(char ***members, int *grmembers, int *i, struct carg *arg, int k; struct passwd *pwd; + if (grp->gr_mem == NULL) + return; + k = 0; while (grp->gr_mem[k] != NULL) { matchFound = false; @@ -415,8 +420,10 @@ print_group(struct group * grp, int pretty) printf("Group Name: %-15s #%lu\n" " Members: ", grp->gr_name, (long) grp->gr_gid); - for (i = 0; grp->gr_mem[i]; i++) - printf("%s%s", i ? "," : "", grp->gr_mem[i]); + if (grp->gr_mem != NULL) { + for (i = 0; grp->gr_mem[i]; i++) + printf("%s%s", i ? "," : "", grp->gr_mem[i]); + } fputs("\n\n", stdout); } return EXIT_SUCCESS; |