From 5fb7a5cf13a3b79edf8673d5db8bf7e8d5e64030 Mon Sep 17 00:00:00 2001 From: Dag-Erling Smørgrav Date: Wed, 9 Apr 2003 18:18:42 +0000 Subject: Avoid clobbering the gecos field, as we intend to use the original later. PR: bin/50563 --- chpass/edit.c | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) (limited to 'chpass/edit.c') diff --git a/chpass/edit.c b/chpass/edit.c index 59646f0..ce82f8e 100644 --- a/chpass/edit.c +++ b/chpass/edit.c @@ -109,7 +109,7 @@ static int display(const char *tfn, struct passwd *pw) { FILE *fp; - char *bp, *p; + char *bp, *gecos, *p; if ((fp = fopen(tfn, "w")) == NULL) { warn("%s", tfn); @@ -151,8 +151,13 @@ display(const char *tfn, struct passwd *pw) (void)fprintf(fp, "Shell: %s\n", *pw->pw_shell ? pw->pw_shell : _PATH_BSHELL); else - list[E_SHELL].restricted = 1; - bp = pw->pw_gecos; + list[E_SHELL].restricted = 1; + + if ((bp = gecos = strdup(pw->pw_gecos)) == NULL) { + warn(NULL); + fclose(fp); + return (-1); + } p = strsep(&bp, ","); p = strdup(p ? p : ""); @@ -183,6 +188,8 @@ display(const char *tfn, struct passwd *pw) if (!list[E_OTHER].restricted || master_mode) (void)fprintf(fp, "Other information: %s\n", bp); + free(gecos); + (void)fchown(fileno(fp), getuid(), getgid()); (void)fclose(fp); return (0); -- cgit v1.2.3-56-ge451