]> git.cameronkatri.com Git - pw-darwin.git/blobdiff - pw/pw_user.c
MF22: Remove confusing comma
[pw-darwin.git] / pw / pw_user.c
index d8a2c2481796544820b35d06e916a908e22b2ba9..d10c5283f5f8ab571f39aa2dcb4489dfb4b5e495 100644 (file)
  * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
  * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
  * SUCH DAMAGE.
- *
- *     $Id: pw_user.c,v 1.1.1.2 1996/12/09 23:55:10 joerg Exp $
  */
 
-#include <unistd.h>
-#include <fcntl.h>
+#ifndef lint
+static const char rcsid[] =
+       "$Id$";
+#endif /* not lint */
+
 #include <ctype.h>
-#include <paths.h>
+#include <err.h>
+#include <fcntl.h>
 #include <sys/param.h>
 #include <dirent.h>
+#include <paths.h>
 #include <termios.h>
+#include <sys/types.h>
+#include <sys/time.h>
+#include <sys/resource.h>
+#include <unistd.h>
+#include <utmp.h>
+#if defined(USE_MD5RAND)
+#include <md5.h>
+#endif
 #include "pw.h"
 #include "bitmap.h"
 #include "pwupd.h"
 
+#if (MAXLOGNAME-1) > UT_NAMESIZE
+#define LOGNAMESIZE UT_NAMESIZE
+#else
+#define LOGNAMESIZE (MAXLOGNAME-1)
+#endif
+
+static          randinit;
+
 static int      print_user(struct passwd * pwd, int pretty);
 static uid_t    pw_uidpolicy(struct userconf * cnf, struct cargs * args);
 static uid_t    pw_gidpolicy(struct userconf * cnf, struct cargs * args, char *nam, gid_t prefer);
@@ -45,9 +64,9 @@ static time_t   pw_exppolicy(struct userconf * cnf, struct cargs * args);
 static char    *pw_homepolicy(struct userconf * cnf, struct cargs * args, char const * user);
 static char    *pw_shellpolicy(struct userconf * cnf, struct cargs * args, char *newshell);
 static char    *pw_password(struct userconf * cnf, struct cargs * args, char const * user);
-static char    *pw_checkname(char *name, int gecos);
 static char    *shell_path(char const * path, char *shells[], char *sh);
 static void     rmat(uid_t uid);
+static void    rmskey(char const * name);
 
 /*-
  * -C config      configuration file
@@ -83,6 +102,7 @@ static void     rmat(uid_t uid);
 int
 pw_user(struct userconf * cnf, int mode, struct cargs * args)
 {
+       int             r, r1;
        char           *p = NULL;
        struct carg    *a_name;
        struct carg    *a_uid;
@@ -90,7 +110,7 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
        struct passwd  *pwd = NULL;
        struct group   *grp;
        struct stat     st;
-       char            line[MAXPWLINE];
+       char            line[_PASSWORD_LEN+1];
 
        static struct passwd fakeuser =
        {
@@ -126,12 +146,70 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
 
        if ((arg = getarg(args, 'b')) != NULL) {
                cnf->home = arg->val;
-               if (stat(cnf->home, &st) == -1 || !S_ISDIR(st.st_mode))
-                       cmderr(EX_OSFILE, "root home `%s' is not a directory or does not exist\n", cnf->home);
        }
+
+       /*
+        * If we'll need to use it or we're updating it,
+        * then create the base home directory if necessary
+        */
+       if (arg != NULL || getarg(args, 'm') != NULL) {
+               int     l = strlen(cnf->home);
+
+               if (l > 1 && cnf->home[l-1] == '/')     /* Shave off any trailing path delimiter */
+                       cnf->home[--l] = '\0';
+
+               if (l < 2 || *cnf->home != '/')         /* Check for absolute path name */
+                       errx(EX_DATAERR, "invalid base directory for home '%s'", cnf->home);
+
+               if (stat(cnf->home, &st) == -1) {
+                       char    dbuf[MAXPATHLEN];
+
+                       /*
+                        * This is a kludge especially for Joerg :)
+                        * If the home directory would be created in the root partition, then
+                        * we really create it under /usr which is likely to have more space.
+                        * But we create a symlink from cnf->home -> "/usr" -> cnf->home
+                        */
+                       if (strchr(cnf->home+1, '/') == NULL) {
+                               strcpy(dbuf, "/usr");
+                               strncat(dbuf, cnf->home, MAXPATHLEN-5);
+                               if (mkdir(dbuf, 0755) != -1 || errno == EEXIST) {
+                                       chown(dbuf, 0, 0);
+                                       symlink(dbuf, cnf->home);
+                               }
+                               /* If this falls, fall back to old method */
+                       }
+                       p = strncpy(dbuf, cnf->home, sizeof dbuf);
+                       dbuf[MAXPATHLEN-1] = '\0';
+                       if (stat(dbuf, &st) == -1) {
+                               while ((p = strchr(++p, '/')) != NULL) {
+                                       *p = '\0';
+                                       if (stat(dbuf, &st) == -1) {
+                                               if (mkdir(dbuf, 0755) == -1)
+                                                       goto direrr;
+                                               chown(dbuf, 0, 0);
+                                       } else if (!S_ISDIR(st.st_mode))
+                                               errx(EX_OSFILE, "'%s' (root home parent) is not a directory", dbuf);
+                                       *p = '/';
+                               }
+                       }
+                       if (stat(dbuf, &st) == -1) {
+                               if (mkdir(dbuf, 0755) == -1) {
+                               direrr: err(EX_OSFILE, "mkdir '%s'", dbuf);
+                               }
+                               chown(dbuf, 0, 0);
+                       }
+               } else if (!S_ISDIR(st.st_mode))
+                       errx(EX_OSFILE, "root home `%s' is not a directory", cnf->home);
+       }
+
+
        if ((arg = getarg(args, 'e')) != NULL)
                cnf->expire_days = atoi(arg->val);
 
+       if ((arg = getarg(args, 'y')) != NULL)
+               cnf->nispasswd = arg->val;
+
        if ((arg = getarg(args, 'p')) != NULL && arg->val)
                cnf->password_days = atoi(arg->val);
 
@@ -139,36 +217,37 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
                p = arg->val;
                if ((grp = getgrnam(p)) == NULL) {
                        if (!isdigit(*p) || (grp = getgrgid((gid_t) atoi(p))) == NULL)
-                               cmderr(EX_NOUSER, "group `%s' does not exist\n", p);
+                               errx(EX_NOUSER, "group `%s' does not exist", p);
                }
                cnf->default_group = newstr(grp->gr_name);
        }
        if ((arg = getarg(args, 'L')) != NULL)
-               cnf->default_class = pw_checkname(arg->val, 0);
+               cnf->default_class = pw_checkname((u_char *)arg->val, 0);
 
        if ((arg = getarg(args, 'G')) != NULL && arg->val) {
                int             i = 0;
 
-               for (p = strtok(arg->val, ", \t"); i < _UC_MAXGROUPS && p != NULL; p = strtok(NULL, ", \t")) {
+               for (p = strtok(arg->val, ", \t"); p != NULL; p = strtok(NULL, ", \t")) {
                        if ((grp = getgrnam(p)) == NULL) {
                                if (!isdigit(*p) || (grp = getgrgid((gid_t) atoi(p))) == NULL)
-                                       cmderr(EX_NOUSER, "group `%s' does not exist\n", p);
+                                       errx(EX_NOUSER, "group `%s' does not exist", p);
                        }
-                       cnf->groups[i++] = newstr(grp->gr_name);
+                       if (extendarray(&cnf->groups, &cnf->numgroups, i + 2) != -1)
+                               cnf->groups[i++] = newstr(grp->gr_name);
                }
-               while (i < _UC_MAXGROUPS)
+               while (i < cnf->numgroups)
                        cnf->groups[i++] = NULL;
        }
        if ((arg = getarg(args, 'k')) != NULL) {
-               if (stat(cnf->dotdir = arg->val, &st) == -1 || S_ISDIR(st.st_mode))
-                       cmderr(EX_OSFILE, "skeleton `%s' is not a directory or does not exist\n", cnf->dotdir);
+               if (stat(cnf->dotdir = arg->val, &st) == -1 || !S_ISDIR(st.st_mode))
+                       errx(EX_OSFILE, "skeleton `%s' is not a directory or does not exist", cnf->dotdir);
        }
        if ((arg = getarg(args, 's')) != NULL)
                cnf->shell_default = arg->val;
 
        if (mode == M_ADD && getarg(args, 'D')) {
                if (getarg(args, 'n') != NULL)
-                       cmderr(EX_DATAERR, "can't combine `-D' with `-n name'\n");
+                       errx(EX_DATAERR, "can't combine `-D' with `-n name'");
                if ((arg = getarg(args, 'u')) != NULL && (p = strtok(arg->val, ", \t")) != NULL) {
                        if ((cnf->min_uid = (uid_t) atoi(p)) == 0)
                                cnf->min_uid = 1000;
@@ -187,7 +266,7 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
                arg = getarg(args, 'C');
                if (write_userconfig(arg ? arg->val : NULL))
                        return EXIT_SUCCESS;
-               perror("config update");
+               warn("config update");
                return EX_IOERR;
        }
        if (mode == M_PRINT && getarg(args, 'a')) {
@@ -200,12 +279,12 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
                return EXIT_SUCCESS;
        }
        if ((a_name = getarg(args, 'n')) != NULL)
-               pwd = getpwnam(pw_checkname(a_name->val, 0));
+               pwd = getpwnam(pw_checkname((u_char *)a_name->val, 0));
        a_uid = getarg(args, 'u');
 
        if (a_uid == NULL) {
                if (a_name == NULL)
-                       cmderr(EX_DATAERR, "user name or id required\n");
+                       errx(EX_DATAERR, "user name or id required");
 
                /*
                 * Determine whether 'n' switch is name or uid - we don't
@@ -231,8 +310,8 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
                                return print_user(&fakeuser, getarg(args, 'P') != NULL);
                        }
                        if (a_name == NULL)
-                               cmderr(EX_NOUSER, "no such uid `%s'\n", a_uid->val);
-                       cmderr(EX_NOUSER, "no such user `%s'\n", a_name->val);
+                               errx(EX_NOUSER, "no such uid `%s'", a_uid->val);
+                       errx(EX_NOUSER, "no such user `%s'", a_name->val);
                }
                if (a_name == NULL)     /* May be needed later */
                        a_name = addarg(args, 'n', newstr(pwd->pw_name));
@@ -246,7 +325,13 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
                        uid_t           uid = pwd->pw_uid;
 
                        if (strcmp(pwd->pw_name, "root") == 0)
-                               cmderr(EX_DATAERR, "cannot remove user 'root'\n");
+                               errx(EX_DATAERR, "cannot remove user 'root'");
+
+                       /*
+                        * Remove skey record from /etc/skeykeys
+                        */
+
+                       rmskey(pwd->pw_name);
 
                        /*
                         * Remove crontabs
@@ -265,7 +350,11 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
                        home[sizeof home - 1] = '\0';
 
                        if (!delpwent(pwd))
-                               cmderr(EX_IOERR, "Error updating passwd file: %s\n", strerror(errno));
+                               err(EX_IOERR, "error updating passwd file");
+
+                       if (cnf->nispasswd && *cnf->nispasswd=='/' && !delnispwent(cnf->nispasswd, a_name->val))
+                               warn("WARNING: NIS passwd update");
+                               
                        editgroups(a_name->val, NULL);
 
                        pw_log(cnf, mode, W_USER, "%s(%ld) account removed", a_name->val, (long) uid);
@@ -301,15 +390,15 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
                 */
                if ((arg = getarg(args, 'l')) != NULL) {
                        if (strcmp(pwd->pw_name, "root") == 0)
-                               cmderr(EX_DATAERR, "can't rename `root' account\n");
-                       pwd->pw_name = pw_checkname(arg->val, 0);
+                               errx(EX_DATAERR, "can't rename `root' account");
+                       pwd->pw_name = pw_checkname((u_char *)arg->val, 0);
                }
                if ((arg = getarg(args, 'u')) != NULL && isdigit(*arg->val)) {
                        pwd->pw_uid = (uid_t) atol(arg->val);
                        if (pwd->pw_uid != 0 && strcmp(pwd->pw_name, "root") == 0)
-                               cmderr(EX_DATAERR, "can't change uid of `root' account\n");
+                               errx(EX_DATAERR, "can't change uid of `root' account");
                        if (pwd->pw_uid == 0 && strcmp(pwd->pw_name, "root") != 0)
-                               fprintf(stderr, "WARNING: account `%s' will have a uid of 0 (superuser access!)\n", pwd->pw_name);
+                               warnx("WARNING: account `%s' will have a uid of 0 (superuser access!)", pwd->pw_name);
                }
                if ((arg = getarg(args, 'g')) != NULL && pwd->pw_uid != 0)      /* Already checked this */
                        pwd->pw_gid = (gid_t) getgrnam(cnf->default_group)->gr_gid;
@@ -322,7 +411,7 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
                                time_t          expire = parse_date(now, arg->val);
 
                                if (now == expire)
-                                       cmderr(EX_DATAERR, "Invalid password change date `%s'\n", arg->val);
+                                       errx(EX_DATAERR, "invalid password change date `%s'", arg->val);
                                pwd->pw_change = expire;
                        }
                }
@@ -334,7 +423,7 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
                                time_t          expire = parse_date(now, arg->val);
 
                                if (now == expire)
-                                       cmderr(EX_DATAERR, "Invalid account expiry date `%s'\n", arg->val);
+                                       errx(EX_DATAERR, "invalid account expiry date `%s'", arg->val);
                                pwd->pw_expire = expire;
                        }
                }
@@ -344,14 +433,22 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
                if (getarg(args, 'L'))
                        pwd->pw_class = cnf->default_class;
 
+               if ((arg  = getarg(args, 'd')) != NULL) {
+                       if (stat(pwd->pw_dir = arg->val, &st) == -1) {
+                               if (getarg(args, 'm') == NULL && strcmp(pwd->pw_dir, "/nonexistent") != 0)
+                                 warnx("WARNING: home `%s' does not exist", pwd->pw_dir);
+                       } else if (!S_ISDIR(st.st_mode))
+                               warnx("WARNING: home `%s' is not a directory", pwd->pw_dir);
+               }
+
                if ((arg = getarg(args, 'w')) != NULL && getarg(args, 'h') == NULL)
                        pwd->pw_passwd = pw_password(cnf, args, pwd->pw_name);
 
        } else {
                if (a_name == NULL)     /* Required */
-                       cmderr(EX_DATAERR, "login name required\n");
+                       errx(EX_DATAERR, "login name required");
                else if ((pwd = getpwnam(a_name->val)) != NULL) /* Exists */
-                       cmderr(EX_DATAERR, "login name `%s' already exists\n", a_name->val);
+                       errx(EX_DATAERR, "login name `%s' already exists", a_name->val);
 
                /*
                 * Now, set up defaults for a new user
@@ -368,14 +465,14 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
                pwd->pw_shell = pw_shellpolicy(cnf, args, NULL);
 
                if (pwd->pw_uid == 0 && strcmp(pwd->pw_name, "root") != 0)
-                       fprintf(stderr, "WARNING: new account `%s' has a uid of 0 (superuser access!)\n", pwd->pw_name);
+                       warnx("WARNING: new account `%s' has a uid of 0 (superuser access!)", pwd->pw_name);
        }
 
        /*
         * Shared add/edit code
         */
        if ((arg = getarg(args, 'c')) != NULL)
-               pwd->pw_gecos = pw_checkname(arg->val, 1);
+               pwd->pw_gecos = pw_checkname((u_char *)arg->val, 1);
 
        if ((arg = getarg(args, 'h')) != NULL) {
                if (strcmp(arg->val, "-") == 0)
@@ -406,14 +503,14 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
                                fflush(stdout);
                        }
                        if (b < 0) {
-                               perror("-h file descriptor");
+                               warn("-h file descriptor");
                                return EX_IOERR;
                        }
                        line[b] = '\0';
                        if ((p = strpbrk(line, " \t\r\n")) != NULL)
                                *p = '\0';
                        if (!*line)
-                               cmderr(EX_DATAERR, "empty password read on file descriptor %d\n", fd);
+                               errx(EX_DATAERR, "empty password read on file descriptor %d", fd);
                        pwd->pw_passwd = pw_pwcrypt(line);
                }
        }
@@ -424,11 +521,25 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
        if (getarg(args, 'N') != NULL)
                return print_user(pwd, getarg(args, 'P') != NULL);
 
-       if ((mode == M_ADD && !addpwent(pwd)) ||
-           (mode == M_UPDATE && !chgpwent(a_name->val, pwd))) {
-               perror("password update");
+       r = r1 = 1;
+       if (mode == M_ADD) {
+               r = addpwent(pwd);
+               if (r && cnf->nispasswd && *cnf->nispasswd=='/')
+                       r1 = addnispwent(cnf->nispasswd, pwd);
+       } else if (mode == M_UPDATE) {
+               r = chgpwent(a_name->val, pwd);
+               if (r && cnf->nispasswd && *cnf->nispasswd=='/')
+                       r1 = chgnispwent(cnf->nispasswd, a_name->val, pwd);
+       }
+
+       if (!r) {
+               warn("password update");
                return EX_IOERR;
+       } else if (!r1) {
+               warn("WARNING: NIS password update");
+               /* Keep on trucking */
        }
+
        /*
         * Ok, user is created or changed - now edit group file
         */
@@ -438,7 +549,7 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
 
        /* pwd may have been invalidated */
        if ((pwd = getpwnam(a_name->val)) == NULL)
-               cmderr(EX_NOUSER, "user '%s' disappeared during update\n", a_name->val);
+               errx(EX_NOUSER, "user '%s' disappeared during update", a_name->val);
 
        grp = getgrgid(pwd->pw_gid);
        pw_log(cnf, mode, W_USER, "%s(%ld):%s(%d):%s:%s:%s",
@@ -466,7 +577,7 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
                        FILE           *pfp = popen(_PATH_SENDMAIL " -t", "w");
 
                        if (pfp == NULL)
-                               perror("sendmail");
+                               warn("sendmail");
                        else {
                                fprintf(pfp, "From: root\n" "To: %s\n" "Subject: Welcome!\n\n", pwd->pw_name);
                                while (fgets(line, sizeof(line), fp) != NULL) {
@@ -508,7 +619,7 @@ pw_uidpolicy(struct userconf * cnf, struct cargs * args)
                uid = (uid_t) atol(a_uid->val);
 
                if ((pwd = getpwuid(uid)) != NULL && getarg(args, 'o') == NULL)
-                       cmderr(EX_DATAERR, "uid `%ld' has already been allocated\n", (long) pwd->pw_uid);
+                       errx(EX_DATAERR, "uid `%ld' has already been allocated", (long) pwd->pw_uid);
        } else {
                struct bitmap   bm;
 
@@ -543,7 +654,7 @@ pw_uidpolicy(struct userconf * cnf, struct cargs * args)
                 * Another sanity check
                 */
                if (uid < cnf->min_uid || uid > cnf->max_uid)
-                       cmderr(EX_SOFTWARE, "unable to allocate a new uid - range fully used\n");
+                       errx(EX_SOFTWARE, "unable to allocate a new uid - range fully used");
                bm_dealloc(&bm);
        }
        return uid;
@@ -571,7 +682,7 @@ pw_gidpolicy(struct userconf * cnf, struct cargs * args, char *nam, gid_t prefer
                if ((grp = getgrnam(a_gid->val)) == NULL) {
                        gid = (gid_t) atol(a_gid->val);
                        if ((gid == 0 && !isdigit(*a_gid->val)) || (grp = getgrgid(gid)) == NULL)
-                               cmderr(EX_NOUSER, "group `%s' is not defined\n", a_gid->val);
+                               errx(EX_NOUSER, "group `%s' is not defined", a_gid->val);
                }
                gid = grp->gr_gid;
        } else if ((grp = getgrnam(nam)) != NULL && grp->gr_mem[0] == NULL) {
@@ -625,11 +736,11 @@ pw_pwdpolicy(struct userconf * cnf, struct cargs * args)
 {
        time_t          result = 0;
        time_t          now = time(NULL);
-       struct carg    *arg = getarg(args, 'e');
+       struct carg    *arg = getarg(args, 'p');
 
        if (arg != NULL) {
                if ((result = parse_date(now, arg->val)) == now)
-                       cmderr(EX_DATAERR, "invalid date/time `%s'\n", arg->val);
+                       errx(EX_DATAERR, "invalid date/time `%s'", arg->val);
        } else if (cnf->password_days > 0)
                result = now + ((long) cnf->password_days * 86400L);
        return result;
@@ -645,7 +756,7 @@ pw_exppolicy(struct userconf * cnf, struct cargs * args)
 
        if (arg != NULL) {
                if ((result = parse_date(now, arg->val)) == now)
-                       cmderr(EX_DATAERR, "invalid date/time `%s'\n", arg->val);
+                       errx(EX_DATAERR, "invalid date/time `%s'", arg->val);
        } else if (cnf->expire_days > 0)
                result = now + ((long) cnf->expire_days * 86400L);
        return result;
@@ -663,7 +774,7 @@ pw_homepolicy(struct userconf * cnf, struct cargs * args, char const * user)
                static char     home[128];
 
                if (cnf->home == NULL || *cnf->home == '\0')
-                       cmderr(EX_CONFIG, "no base home directory set\n");
+                       errx(EX_CONFIG, "no base home directory set");
                sprintf(home, "%s/%s", cnf->home, user);
                return home;
        }
@@ -699,8 +810,8 @@ shell_path(char const * path, char *shells[], char *sh)
                                }
                }
                if (sh == NULL)
-                       cmderr(EX_OSFILE, "can't find shell `%s' in shell paths\n", sh);
-               cmderr(EX_CONFIG, "no default shell available or defined\n");
+                       errx(EX_OSFILE, "can't find shell `%s' in shell paths", sh);
+               errx(EX_CONFIG, "no default shell available or defined");
                return NULL;
        }
 }
@@ -730,7 +841,14 @@ pw_pwcrypt(char *password)
        /*
         * Calculate a salt value
         */
-       srandom((unsigned) (time(NULL) | getpid()));
+       if (!randinit) {
+               randinit = 1;
+#ifdef __FreeBSD__
+               srandomdev();
+#else
+               srandom((unsigned long) (time(NULL) ^ getpid()));
+#endif
+       }
        for (i = 0; i < 8; i++)
                salt[i] = chars[random() % 63];
        salt[i] = '\0';
@@ -738,27 +856,84 @@ pw_pwcrypt(char *password)
        return strcpy(buf, crypt(password, salt));
 }
 
+#if defined(USE_MD5RAND)
+u_char *
+pw_getrand(u_char *buf, int len)       /* cryptographically secure rng */
+{
+       int i;
+       for (i=0;i<len;i+=16) {
+               u_char ubuf[16];
+
+               MD5_CTX md5_ctx;
+               struct timeval tv, tvo;
+               struct rusage ru;
+               int n=0;
+               int t;
+
+               MD5Init (&md5_ctx);
+               t=getpid();
+               MD5Update (&md5_ctx, (u_char*)&t, sizeof t);
+               t=getppid();
+               MD5Update (&md5_ctx, (u_char*)&t, sizeof t);
+               gettimeofday (&tvo, NULL);
+               do {
+                       getrusage (RUSAGE_SELF, &ru);
+                       MD5Update (&md5_ctx, (u_char*)&ru, sizeof ru);
+                       gettimeofday (&tv, NULL);
+                       MD5Update (&md5_ctx, (u_char*)&tv, sizeof tv);
+               } while (n++<20 || tv.tv_usec-tvo.tv_usec<100*1000);
+               MD5Final (ubuf, &md5_ctx);
+               memcpy(buf+i, ubuf, MIN(16, len-n));
+       }
+       return buf;
+}
+
+#else  /* Portable version */
+
+static u_char *
+pw_getrand(u_char *buf, int len)
+{
+       int i;
+
+       for (i = 0; i < len; i++) {
+               unsigned long val = random();
+               /* Use all bits in the random value */
+               buf[i]=(u_char)((val >> 24) ^ (val >> 16) ^ (val >> 8) ^ val);
+       }
+       return buf;
+}
+
+#endif
 
 static char    *
 pw_password(struct userconf * cnf, struct cargs * args, char const * user)
 {
        int             i, l;
        char            pwbuf[32];
+       u_char          rndbuf[sizeof pwbuf];
 
        switch (cnf->default_password) {
        case -1:                /* Random password */
-               srandom((unsigned) (time(NULL) | getpid()));
+               if (!randinit) {
+                       randinit = 1;
+#ifdef __FreeBSD__
+                       srandomdev();
+#else
+                       srandom((unsigned long) (time(NULL) ^ getpid()));
+#endif
+               }
                l = (random() % 8 + 8); /* 8 - 16 chars */
+               pw_getrand(rndbuf, l);
                for (i = 0; i < l; i++)
-                       pwbuf[i] = chars[random() % sizeof(chars)];
+                       pwbuf[i] = chars[rndbuf[i] % sizeof(chars)];
                pwbuf[i] = '\0';
 
                /*
                 * We give this information back to the user
                 */
                if (getarg(args, 'h') == NULL && getarg(args, 'N') == NULL) {
-                       if (isatty(0))
-                               printf("Password is: ");
+                       if (isatty(1))
+                               printf("Password for '%s' is: ", user);
                        printf("%s\n", pwbuf);
                        fflush(stdout);
                }
@@ -794,6 +969,8 @@ print_user(struct passwd * pwd, int pretty)
                struct group   *grp = getgrgid(pwd->pw_gid);
                char            uname[60] = "User &", office[60] = "[None]",
                                wphone[60] = "[None]", hphone[60] = "[None]";
+               char            acexpire[32] = "[None]", pwexpire[32] = "[None]";
+               struct tm *    tptr;
 
                if ((p = strtok(pwd->pw_gecos, ",")) != NULL) {
                        strncpy(uname, p, sizeof uname);
@@ -822,26 +999,31 @@ print_user(struct passwd * pwd, int pretty)
                        memmove(p, pwd->pw_name, l);
                        *p = (char) toupper(*p);
                }
-               printf("Login Name : %-10s   #%-22ld  Group : %-10s   #%ld\n"
-                      " Full Name : %s\n"
-                      "      Home : %-32.32s      Class : %s\n"
-                      "     Shell : %-32.32s     Office : %s\n"
-                      "Work Phone : %-32.32s Home Phone : %s\n",
-                      
+               if (pwd->pw_expire > (time_t)0 && (tptr = localtime(&pwd->pw_expire)) != NULL)
+                 strftime(acexpire, sizeof acexpire, "%c", tptr);
+               if (pwd->pw_change > (time_t)9 && (tptr = localtime(&pwd->pw_change)) != NULL)
+                 strftime(pwexpire, sizeof pwexpire, "%c", tptr);
+               printf("Login Name: %-15s   #%-12ld Group: %-15s   #%ld\n"
+                      " Full Name: %s\n"
+                      "      Home: %-26.26s      Class: %s\n"
+                      "     Shell: %-26.26s     Office: %s\n"
+                      "Work Phone: %-26.26s Home Phone: %s\n"
+                      "Acc Expire: %-26.26s Pwd Expire: %s\n",
                       pwd->pw_name, (long) pwd->pw_uid,
                       grp ? grp->gr_name : "(invalid)", (long) pwd->pw_gid,
                       uname, pwd->pw_dir, pwd->pw_class,
-                      pwd->pw_shell, office, wphone, hphone);
+                      pwd->pw_shell, office, wphone, hphone,
+                      acexpire, pwexpire);
                setgrent();
                j = 0;
                while ((grp=getgrent()) != NULL)
                {
                        int     i = 0;
-                       while (i < _UC_MAXGROUPS && grp->gr_mem[i] != NULL)
+                       while (grp->gr_mem[i] != NULL)
                        {
                                if (strcmp(grp->gr_mem[i], pwd->pw_name)==0)
                                {
-                                       printf(j++ == 0 ? "    Groups : %s" : ",%s", grp->gr_name);
+                                       printf(j++ == 0 ? "    Groups: %s" : ",%s", grp->gr_name);
                                        break;
                                }
                                ++i;
@@ -853,20 +1035,25 @@ print_user(struct passwd * pwd, int pretty)
        return EXIT_SUCCESS;
 }
 
-static char    *
-pw_checkname(char *name, int gecos)
+char    *
+pw_checkname(u_char *name, int gecos)
 {
        int             l = 0;
-       char const     *notch = gecos ? ":" : " :+-&#%$^()!@~*?<>=|\\/\"";
+       char const     *notch = gecos ? ":!@" : " ,\t:+&#%$^()!@~*?<>=|\\/\"";
 
        while (name[l]) {
-               if (strchr(notch, name[l]) != NULL || name[l] < ' ')
-                       cmderr(EX_DATAERR, "invalid character `%c' in field\n", name[l]);
+               if (strchr(notch, name[l]) != NULL || name[l] < ' ' || name[l] == 127 ||
+                       (!gecos && l==0 && name[l] == '-') ||   /* leading '-' */
+                       (!gecos && name[l] & 0x80))     /* 8-bit */
+                       errx(EX_DATAERR, (name[l] >= ' ' && name[l] < 127)
+                                           ? "invalid character `%c' in field"
+                                           : "invalid character 0x%02x in field",
+                                           name[l]);
                ++l;
        }
-       if (!gecos && l > 8)
-               cmderr(EX_DATAERR, "name too long `%s'\n", name);
-       return name;
+       if (!gecos && l > LOGNAMESIZE)
+               errx(EX_DATAERR, "name too long `%s'", name);
+       return (char *)name;
 }
 
 
@@ -894,3 +1081,31 @@ rmat(uid_t uid)
                closedir(d);
        }
 }
+
+static void
+rmskey(char const * name)
+{
+       static const char etcskey[] = "/etc/skeykeys";
+       FILE   *fp = fopen(etcskey, "r+");
+
+       if (fp != NULL) {
+               char    tmp[1024];
+               off_t   atofs = 0;
+               int     length = strlen(name);
+
+               while (fgets(tmp, sizeof tmp, fp) != NULL) {
+                       if (strncmp(name, tmp, length) == 0 && tmp[length]==' ') {
+                               if (fseek(fp, atofs, SEEK_SET) == 0) {
+                                       fwrite("#", 1, 1, fp);  /* Comment username out */
+                               }
+                               break;
+                       }
+                       atofs = ftell(fp);
+               }
+               /*
+                * If we got an error of any sort, don't update!
+                */
+               fclose(fp);
+       }
+}
+