]> git.cameronkatri.com Git - pw-darwin.git/blobdiff - pw/pw_user.c
Cast -1 to uid_t and bump WARNING level to 3
[pw-darwin.git] / pw / pw_user.c
index 1b72cbdb70eef84d1cefdd475d63039ad5078aad..112d0183a5e1cc9c61b4680060f095f3a5f96ec0 100644 (file)
@@ -40,8 +40,10 @@ static const char rcsid[] =
 #include <sys/types.h>
 #include <sys/time.h>
 #include <sys/resource.h>
-#include <unistd.h>
 #include <login_cap.h>
+#include <pwd.h>
+#include <grp.h>
+#include <libutil.h>
 #include "pw.h"
 #include "bitmap.h"
 
@@ -182,8 +184,7 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
                         * 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);
+                               snprintf(dbuf, MAXPATHLEN, "/usr%s", cnf->home);
                                if (mkdir(dbuf, _DEF_DIRMODE) != -1 || errno == EEXIST) {
                                        chown(dbuf, 0, 0);
                                        /*
@@ -197,7 +198,7 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
                        strlcpy(dbuf, cnf->home, sizeof(dbuf));
                        p = dbuf;
                        if (stat(dbuf, &st) == -1) {
-                               while ((p = strchr(++p, '/')) != NULL) {
+                               while ((p = strchr(p + 1, '/')) != NULL) {
                                        *p = '\0';
                                        if (stat(dbuf, &st) == -1) {
                                                if (mkdir(dbuf, _DEF_DIRMODE) == -1)
@@ -285,14 +286,12 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
                arg = getarg(args, 'C');
                if (write_userconfig(arg ? arg->val : NULL))
                        return EXIT_SUCCESS;
-               warn("config update");
-               return EX_IOERR;
+               err(EX_IOERR, "config udpate");
        }
 
        if (mode == M_PRINT && getarg(args, 'a')) {
                int             pretty = getarg(args, 'P') != NULL;
                int             v7 = getarg(args, '7') != NULL;
-
                SETPWENT();
                while ((pwd = GETPWENT()) != NULL)
                        print_user(pwd, pretty, v7);
@@ -319,6 +318,9 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
                        (a_uid = a_name)->ch = 'u';
                        a_name = NULL;
                }
+       } else {
+               if (strspn(a_uid->val, "0123456789") != strlen(a_uid->val))
+                       errx(EX_USAGE, "-u expects a number");
        }
 
        /*
@@ -333,7 +335,7 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
                if (pwd == NULL) {
                        if (mode == M_PRINT && getarg(args, 'F')) {
                                fakeuser.pw_name = a_name ? a_name->val : "nouser";
-                               fakeuser.pw_uid = a_uid ? (uid_t) atol(a_uid->val) : -1;
+                               fakeuser.pw_uid = a_uid ? (uid_t) atol(a_uid->val) : (uid_t) -1;
                                return print_user(&fakeuser,
                                                  getarg(args, 'P') != NULL,
                                                  getarg(args, '7') != NULL);
@@ -359,11 +361,9 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
                if (mode == M_LOCK) {
                        if (strncmp(pwd->pw_passwd, locked_str, sizeof(locked_str)-1) == 0)
                                errx(EX_DATAERR, "user '%s' is already locked", pwd->pw_name);
-                       passtmp = malloc(strlen(pwd->pw_passwd) + sizeof(locked_str));
+                       asprintf(&passtmp, "%s%s", locked_str, pwd->pw_passwd);
                        if (passtmp == NULL)    /* disaster */
                                errx(EX_UNAVAILABLE, "out of memory");
-                       strcpy(passtmp, locked_str);
-                       strcat(passtmp, pwd->pw_passwd);
                        pwd->pw_passwd = passtmp;
                        edited = 1;
                } else if (mode == M_UNLOCK) {
@@ -378,6 +378,8 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
                        char            file[MAXPATHLEN];
                        char            home[MAXPATHLEN];
                        uid_t           uid = pwd->pw_uid;
+                       struct group    *gr;
+                       char            grname[LOGNAMESIZE];
 
                        if (strcmp(pwd->pw_name, "root") == 0)
                                errx(EX_DATAERR, "cannot remove user 'root'");
@@ -392,9 +394,9 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
                                /*
                                 * Remove crontabs
                                 */
-                               sprintf(file, "/var/cron/tabs/%s", pwd->pw_name);
+                               snprintf(file, sizeof(file), "/var/cron/tabs/%s", pwd->pw_name);
                                if (access(file, F_OK) == 0) {
-                                       sprintf(file, "crontab -u %s -r", pwd->pw_name);
+                                       snprintf(file, sizeof(file), "crontab -u %s -r", pwd->pw_name);
                                        system(file);
                                }
                        }
@@ -402,16 +404,19 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
                         * Save these for later, since contents of pwd may be
                         * invalidated by deletion
                         */
-                       sprintf(file, "%s/%s", _PATH_MAILDIR, pwd->pw_name);
+                       snprintf(file, sizeof(file), "%s/%s", _PATH_MAILDIR, pwd->pw_name);
                        strlcpy(home, pwd->pw_dir, sizeof(home));
+                       gr = GETGRGID(pwd->pw_gid);
+                       if (gr != NULL)
+                               strlcpy(grname, gr->gr_name, LOGNAMESIZE);
+                       else
+                               grname[0] = '\0';
 
                        rc = delpwent(pwd);
                        if (rc == -1)
                                err(EX_IOERR, "user '%s' does not exist", pwd->pw_name);
-                       else if (rc != 0) {
-                               warn("passwd update");
-                               return EX_IOERR;
-                       }
+                       else if (rc != 0)
+                               err(EX_IOERR, "passwd update");
 
                        if (cnf->nispasswd && *cnf->nispasswd=='/') {
                                rc = delnispwent(cnf->nispasswd, a_name->val);
@@ -422,7 +427,27 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
                                /* non-fatal */
                        }
 
-                       editgroups(a_name->val, NULL);
+                       grp = GETGRNAM(a_name->val);
+                       if (grp != NULL &&
+                           (grp->gr_mem == NULL || *grp->gr_mem == NULL) &&
+                           strcmp(a_name->val, grname) == 0)
+                               delgrent(GETGRNAM(a_name->val));
+                       SETGRENT();
+                       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);
+                                               }
+                                       }
+                               }
+                       }
+                       ENDGRENT();
 
                        pw_log(cnf, mode, W_USER, "%s(%ld) account removed", a_name->val, (long) uid);
 
@@ -494,8 +519,6 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
                                time_t          now = time(NULL);
                                time_t          expire = parse_date(now, arg->val);
 
-                               if (now == expire)
-                                       errx(EX_DATAERR, "invalid password change date `%s'", arg->val);
                                if (pwd->pw_change != expire) {
                                        pwd->pw_change = expire;
                                        edited = 1;
@@ -514,8 +537,6 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
                                time_t          now = time(NULL);
                                time_t          expire = parse_date(now, arg->val);
 
-                               if (now == expire)
-                                       errx(EX_DATAERR, "invalid account expiry date `%s'", arg->val);
                                if (pwd->pw_expire != expire) {
                                        pwd->pw_expire = expire;
                                        edited = 1;
@@ -558,7 +579,7 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
 
                        lc = login_getpwclass(pwd);
                        if (lc == NULL ||
-                           login_setcryptfmt(lc, "md5", NULL) == NULL)
+                           login_setcryptfmt(lc, "sha512", NULL) == NULL)
                                warn("setting crypt(3) format");
                        login_close(lc);
                        pwd->pw_passwd = pw_password(cnf, args, pwd->pw_name);
@@ -590,7 +611,7 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
                pwd->pw_dir = pw_homepolicy(cnf, args, pwd->pw_name);
                pwd->pw_shell = pw_shellpolicy(cnf, args, NULL);
                lc = login_getpwclass(pwd);
-               if (lc == NULL || login_setcryptfmt(lc, "md5", NULL) == NULL)
+               if (lc == NULL || login_setcryptfmt(lc, "sha512", NULL) == NULL)
                        warn("setting crypt(3) format");
                login_close(lc);
                pwd->pw_passwd = pw_password(cnf, args, pwd->pw_name);
@@ -648,11 +669,9 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
                                fputc('\n', stdout);
                                fflush(stdout);
                        }
-                       if (b < 0) {
-                               warn("-%c file descriptor", precrypt ? 'H' :
-                                   'h');
-                               return EX_IOERR;
-                       }
+                       if (b < 0)
+                               err(EX_IOERR, "-%c file descriptor",
+                                   precrypt ? 'H' : 'h');
                        line[b] = '\0';
                        if ((p = strpbrk(line, "\r\n")) != NULL)
                                *p = '\0';
@@ -665,7 +684,7 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
                        } else {
                                lc = login_getpwclass(pwd);
                                if (lc == NULL ||
-                                   login_setcryptfmt(lc, "md5", NULL) == NULL)
+                                   login_setcryptfmt(lc, "sha512", NULL) == NULL)
                                        warn("setting crypt(3) format");
                                login_close(lc);
                                pwd->pw_passwd = pw_pwcrypt(line);
@@ -685,13 +704,11 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
        if (mode == M_ADD) {
                edited = 1;     /* Always */
                rc = addpwent(pwd);
-               if (rc == -1) {
-                       warnx("user '%s' already exists", pwd->pw_name);
-                       return EX_IOERR;
-               } else if (rc != 0) {
-                       warn("passwd file update");
-                       return EX_IOERR;
-               }
+               if (rc == -1)
+                       errx(EX_IOERR, "user '%s' already exists",
+                           pwd->pw_name);
+               else if (rc != 0)
+                       err(EX_IOERR, "passwd file update");
                if (cnf->nispasswd && *cnf->nispasswd=='/') {
                        rc = addnispwent(cnf->nispasswd, pwd);
                        if (rc == -1)
@@ -703,13 +720,10 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
        } else if (mode == M_UPDATE || mode == M_LOCK || mode == M_UNLOCK) {
                if (edited) {   /* Only updated this if required */
                        rc = chgpwent(a_name->val, pwd);
-                       if (rc == -1) {
-                               warnx("user '%s' does not exist (NIS?)", pwd->pw_name);
-                               return EX_IOERR;
-                       } else if (rc != 0) {
-                               warn("passwd file update");
-                               return EX_IOERR;
-                       }
+                       if (rc == -1)
+                               errx(EX_IOERR, "user '%s' does not exist (NIS?)", pwd->pw_name);
+                       else if (rc != 0)
+                               err(EX_IOERR, "passwd file update");
                        if ( cnf->nispasswd && *cnf->nispasswd=='/') {
                                rc = chgnispwent(cnf->nispasswd, a_name->val, pwd);
                                if (rc == -1)
@@ -725,8 +739,42 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
         * Ok, user is created or changed - now edit group file
         */
 
-       if (mode == M_ADD || getarg(args, 'G') != NULL)
-               editgroups(pwd->pw_name, cnf->groups);
+       if (mode == M_ADD || getarg(args, 'G') != NULL) {
+               int i, j;
+               /* First remove the user from all group */
+               SETGRENT();
+               while ((grp = GETGRENT()) != NULL) {
+                       char group[MAXLOGNAME];
+                       if (grp->gr_mem == NULL)
+                               continue;
+                       for (i = 0; grp->gr_mem[i] != NULL; i++) {
+                               if (strcmp(grp->gr_mem[i] , pwd->pw_name) != 0)
+                                       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();
+
+               /* now add to group where needed */
+               for (i = 0; cnf->groups[i] != NULL; i++) {
+                       grp = GETGRNAM(cnf->groups[i]);
+                       grp = gr_add(grp, pwd->pw_name);
+                       /*
+                        * grp can only be NULL in 2 cases:
+                        * - the new member is already a member
+                        * - a problem with memory occurs
+                        * in both cases we want to skip now.
+                        */
+                       if (grp == NULL)
+                               continue;
+                       chggrent(cnf->groups[i], grp);
+                       free(grp);
+               }
+       }
+
 
        /* go get a current version of pwd */
        pwd = GETPWNAM(a_name->val);
@@ -743,7 +791,7 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
        grp = GETGRGID(pwd->pw_gid);
        pw_log(cnf, mode, W_USER, "%s(%ld):%s(%ld):%s:%s:%s",
               pwd->pw_name, (long) pwd->pw_uid,
-           grp ? grp->gr_name : "unknown", (long) (grp ? grp->gr_gid : -1),
+           grp ? grp->gr_name : "unknown", (long) (grp ? grp->gr_gid : (uid_t)-1),
               pwd->pw_gecos, pwd->pw_dir, pwd->pw_shell);
 
        /*
@@ -753,7 +801,7 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args)
         */
        if (mode == M_ADD) {
                if (!PWALTDIR()) {
-                       sprintf(line, "%s/%s", _PATH_MAILDIR, pwd->pw_name);
+                       snprintf(line, sizeof(line), "%s/%s", _PATH_MAILDIR, pwd->pw_name);
                        close(open(line, O_RDWR | O_CREAT, 0600));      /* Preserve contents &
                                                                         * mtime */
                        chown(line, pwd->pw_uid, pwd->pw_gid);
@@ -877,7 +925,8 @@ pw_gidpolicy(struct userconf * cnf, struct cargs * args, char *nam, gid_t prefer
                                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) {
+       } else if ((grp = GETGRNAM(nam)) != NULL &&
+           (grp->gr_mem == NULL || grp->gr_mem[0] == NULL)) {
                gid = grp->gr_gid;  /* Already created? Use it anyway... */
        } else {
                struct cargs    grpargs;
@@ -896,7 +945,7 @@ pw_gidpolicy(struct userconf * cnf, struct cargs * args, char *nam, gid_t prefer
                 * function will happily handle that case for us and exit.
                 */
                if (GETGRGID(prefer) == NULL) {
-                       sprintf(tmp, "%lu", (unsigned long) prefer);
+                       snprintf(tmp, sizeof(tmp), "%u", prefer);
                        addarg(&grpargs, 'g', tmp);
                }
                if (getarg(args, 'N'))
@@ -959,17 +1008,16 @@ static char    *
 pw_homepolicy(struct userconf * cnf, struct cargs * args, char const * user)
 {
        struct carg    *arg = getarg(args, 'd');
+       static char     home[128];
 
        if (arg)
-               return arg->val;
-       else {
-               static char     home[128];
+               return (arg->val);
 
-               if (cnf->home == NULL || *cnf->home == '\0')
-                       errx(EX_CONFIG, "no base home directory set");
-               sprintf(home, "%s/%s", cnf->home, user);
-               return home;
-       }
+       if (cnf->home == NULL || *cnf->home == '\0')
+               errx(EX_CONFIG, "no base home directory set");
+       snprintf(home, sizeof(home), "%s/%s", cnf->home, user);
+
+       return (home);
 }
 
 static char    *
@@ -990,12 +1038,12 @@ shell_path(char const * path, char *shells[], char *sh)
                        static char     shellpath[256];
 
                        if (sh != NULL) {
-                               sprintf(shellpath, "%s/%s", p, sh);
+                               snprintf(shellpath, sizeof(shellpath), "%s/%s", p, sh);
                                if (access(shellpath, X_OK) == 0)
                                        return shellpath;
                        } else
                                for (i = 0; i < _UC_MAXSHELLS && shells[i] != NULL; i++) {
-                                       sprintf(shellpath, "%s/%s", p, shells[i]);
+                                       snprintf(shellpath, sizeof(shellpath), "%s/%s", p, shells[i]);
                                        if (access(shellpath, X_OK) == 0)
                                                return shellpath;
                                }
@@ -1090,10 +1138,14 @@ static int
 print_user(struct passwd * pwd, int pretty, int v7)
 {
        if (!pretty) {
-               char            buf[_UC_MAXLINE];
+               char            *buf;
+
+               if (!v7)
+                       pwd->pw_passwd = (pwd->pw_passwd == NULL) ? "" : "*";
 
-               fmtpwentry(buf, pwd, v7 ? PWF_PASSWD : PWF_STANDARD);
-               fputs(buf, stdout);
+               buf = v7 ? pw_make_v7(pwd) : pw_make(pwd);
+               printf("%s\n", buf);
+               free(buf);
        } else {
                int             j;
                char           *p;
@@ -1147,14 +1199,16 @@ print_user(struct passwd * pwd, int pretty, int v7)
                while ((grp=GETGRENT()) != NULL)
                {
                        int     i = 0;
-                       while (grp->gr_mem[i] != NULL)
-                       {
-                               if (strcmp(grp->gr_mem[i], pwd->pw_name)==0)
+                       if (grp->gr_mem != NULL) {
+                               while (grp->gr_mem[i] != NULL)
                                {
-                                       printf(j++ == 0 ? "    Groups: %s" : ",%s", grp->gr_name);
-                                       break;
+                                       if (strcmp(grp->gr_mem[i], pwd->pw_name)==0)
+                                       {
+                                               printf(j++ == 0 ? "    Groups: %s" : ",%s", grp->gr_name);
+                                               break;
+                                       }
+                                       ++i;
                                }
-                               ++i;
                        }
                }
                ENDGRENT();
@@ -1239,7 +1293,7 @@ rmat(uid_t uid)
                            st.st_uid == uid) {
                                char            tmp[MAXPATHLEN];
 
-                               sprintf(tmp, "/usr/bin/atrm %s", e->d_name);
+                               snprintf(tmp, sizeof(tmp), "/usr/bin/atrm %s", e->d_name);
                                system(tmp);
                        }
                }