Imported Upstream version 1.8.7
[debian/sudo] / plugins / sudoers / sudoers.c
index f533e591bd6b0483e79942197fd4cb1fc105392a..c06907cea6dcdbc05f96fb0f88edcc4766f1602a 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 1993-1996, 1998-2011 Todd C. Miller <Todd.Miller@courtesan.com>
+ * Copyright (c) 1993-1996, 1998-2013 Todd C. Miller <Todd.Miller@courtesan.com>
  *
  * Permission to use, copy, modify, and distribute this software for any
  * purpose with or without fee is hereby granted, provided that the above
@@ -16,9 +16,6 @@
  * Sponsored in part by the Defense Advanced Research Projects
  * Agency (DARPA) and Air Force Research Laboratory, Air Force
  * Materiel Command, USAF, under agreement number F39502-99-1-0512.
- *
- * For a brief history of sudo, please see the HISTORY file included
- * with this distribution.
  */
 
 #define _SUDO_MAIN
@@ -31,7 +28,6 @@
 
 #include <sys/types.h>
 #include <sys/stat.h>
-#include <sys/param.h>
 #include <sys/socket.h>
 #include <stdio.h>
 #ifdef STDC_HEADERS
 #include <signal.h>
 #include <grp.h>
 #include <time.h>
-#ifdef HAVE_SETLOCALE
-# include <locale.h>
-#endif
-#include <netinet/in.h>
 #include <netdb.h>
 #ifdef HAVE_LOGIN_CAP_H
 # include <login_cap.h>
 # ifndef LOGIN_DEFROOTCLASS
 #  define LOGIN_DEFROOTCLASS   "daemon"
 # endif
+# ifndef LOGIN_SETENV
+#  define LOGIN_SETENV 0
+# endif
 #endif
 #ifdef HAVE_SELINUX
 # include <selinux/selinux.h>
 #endif
-#ifdef HAVE_MBR_CHECK_MEMBERSHIP
-# include <membership.h>
-#endif
 #include <ctype.h>
-#include <setjmp.h>
+#ifndef HAVE_GETADDRINFO
+# include "compat/getaddrinfo.h"
+#endif
 
 #include "sudoers.h"
-#include "lbuf.h"
-#include "interfaces.h"
-#include "sudoers_version.h"
 #include "auth/sudo_auth.h"
+#include "secure_path.h"
 
 /*
  * Prototypes
  */
-static void init_vars(char * const *);
-static int set_cmnd(int);
-static void set_loginclass(struct passwd *);
-static void set_runasgr(char *);
-static void set_runaspw(char *);
-static int sudoers_policy_version(int verbose);
-static int deserialize_info(char * const settings[], char * const user_info[]);
 static char *find_editor(int nfiles, char **files, char ***argv_out);
+static int cb_runas_default(const char *);
+static int cb_sudoers_locale(const char *);
+static int set_cmnd(void);
 static void create_admin_success_flag(void);
-
-/* XXX */
-extern int runas_ngroups;
-extern GETGROUPS_T *runas_groups;
+static void init_vars(char * const *);
+static void set_fqdn(void);
+static void set_loginclass(struct passwd *);
+static void set_runasgr(const char *);
+static void set_runaspw(const char *);
+static bool tty_present(void);
 
 /*
  * Globals
  */
-const char *sudoers_file = _PATH_SUDOERS;
-mode_t sudoers_mode = SUDOERS_MODE;
-uid_t sudoers_uid = SUDOERS_UID;
-gid_t sudoers_gid = SUDOERS_GID;
 struct sudo_user sudo_user;
 struct passwd *list_pw;
-struct interface *interfaces;
 int long_list;
-int debug_level;
 uid_t timestamp_uid;
 extern int errorlineno;
-extern int parse_error;
+extern bool parse_error;
 extern char *errorfile;
-#ifdef HAVE_LOGIN_CAP_H
-login_cap_t *lc;
-#endif /* HAVE_LOGIN_CAP_H */
 #ifdef HAVE_BSD_AUTH_H
 char *login_style;
 #endif /* HAVE_BSD_AUTH_H */
-sudo_conv_t sudo_conv;
-sudo_printf_t sudo_printf;
 int sudo_mode;
 
 static char *prev_user;
 static char *runas_user;
 static char *runas_group;
 static struct sudo_nss_list *snl;
-static const char *interfaces_string;
-static sigaction_t saved_sa_int, saved_sa_quit, saved_sa_tstp;
 
 /* XXX - must be extern for audit bits of sudo_auth.c */
 int NewArgc;
 char **NewArgv;
 
-/* plugin_error.c */
-extern sigjmp_buf error_jmp;
-
-static int
-sudoers_policy_open(unsigned int version, sudo_conv_t conversation,
-    sudo_printf_t plugin_printf, char * const settings[],
-    char * const user_info[], char * const envp[])
+int
+sudoers_policy_init(void *info, char * const envp[])
 {
     volatile int sources = 0;
-    sigaction_t sa;
-    struct sudo_nss *nss;
-
-    if (!sudo_conv)
-       sudo_conv = conversation;
-    if (!sudo_printf)
-       sudo_printf = plugin_printf;
+    struct sudo_nss *nss, *nss_next;
+    debug_decl(sudoers_policy_init, SUDO_DEBUG_PLUGIN)
 
-    if (sigsetjmp(error_jmp, 1)) {
-       /* called via error(), errorx() or log_error() */
-       rewind_perms();
-       return -1;
-    }
-
-    /*
-     * Signal setup:
-     * Ignore keyboard-generated signals so the user cannot interrupt
-     *  us at some point and avoid the logging.
-     *  Install handler to wait for children when they exit.
-     */
-    zero_bytes(&sa, sizeof(sa));
-    sigemptyset(&sa.sa_mask);
-    sa.sa_flags = SA_RESTART;
-    sa.sa_handler = SIG_IGN;
-    (void) sigaction(SIGINT, &sa, &saved_sa_int);
-    (void) sigaction(SIGQUIT, &sa, &saved_sa_quit);
-    (void) sigaction(SIGTSTP, &sa, &saved_sa_tstp);
+    bindtextdomain("sudoers", LOCALEDIR);
 
     sudo_setpwent();
     sudo_setgrent();
 
+    /* Register fatal/fatalx callback. */
+    fatal_callback_register(sudoers_cleanup);
+
     /* Initialize environment functions (including replacements). */
     env_init(envp);
 
     /* Setup defaults data structures. */
     init_defaults();
 
-    /* Parse settings and user_info */
-    sudo_mode = deserialize_info(settings, user_info);
+    /* Parse info from front-end. */
+    sudo_mode = sudoers_policy_deserialize_info(info, &runas_user, &runas_group);
 
     init_vars(envp);           /* XXX - move this later? */
 
     /* Parse nsswitch.conf for sudoers order. */
     snl = sudo_read_nss();
 
+    /* LDAP or NSS may modify the euid so we need to be root for the open. */
     set_perms(PERM_INITIAL);
+    set_perms(PERM_ROOT);
 
     /* Open and parse sudoers, set global defaults */
-    tq_foreach_fwd(snl, nss) {
-       if (nss->open(nss) == 0 && nss->parse(nss) == 0) {
-           sources++;
-           if (nss->setdefs(nss) != 0)
-               log_error(NO_STDERR|NO_EXIT, "problem with defaults entries");
-       }
+    for (nss = snl->first; nss != NULL; nss = nss_next) {
+        nss_next = nss->next;
+        if (nss->open(nss) == 0 && nss->parse(nss) == 0) {
+            sources++;
+            if (nss->setdefs(nss) != 0)
+                log_warning(NO_STDERR, N_("problem with defaults entries"));
+        } else {
+            tq_remove(snl, nss);
+        }
     }
     if (sources == 0) {
-       warningx("no valid sudoers sources found, quitting");
-       return -1;
+       warningx(_("no valid sudoers sources found, quitting"));
+       debug_return_bool(-1);
     }
 
     /* XXX - collect post-sudoers parse settings into a function */
 
     /*
-     * Initialize external group plugin.
+     * Initialize external group plugin, if any.
      */
     if (def_group_plugin) {
-       switch (group_plugin_load(def_group_plugin)) {
-       case -1:
-           return -1;
-       case FALSE:
+       if (group_plugin_load(def_group_plugin) != true)
            def_group_plugin = NULL;
-       }
     }
 
     /*
@@ -228,6 +181,7 @@ sudoers_policy_open(unsigned int version, sudo_conv_t conversation,
      * Note that if runas_group was specified without runas_user we
      * defer setting runas_pw so the match routines know to ignore it.
      */
+    /* XXX - qpm4u does more here as it may have already set runas_pw */
     if (runas_group != NULL) {
        set_runasgr(runas_group);
        if (runas_user != NULL)
@@ -236,84 +190,48 @@ sudoers_policy_open(unsigned int version, sudo_conv_t conversation,
        set_runaspw(runas_user ? runas_user : def_runas_default);
 
     if (!update_defaults(SETDEF_RUNAS))
-       log_error(NO_STDERR|NO_EXIT, "problem with defaults entries");
+       log_warning(NO_STDERR, N_("problem with defaults entries"));
 
     if (def_fqdn)
        set_fqdn();     /* deferred until after sudoers is parsed */
 
     /* Set login class if applicable. */
-    set_loginclass(sudo_user.pw);
+    set_loginclass(runas_pw ? runas_pw : sudo_user.pw);
 
     restore_perms();
 
-    return TRUE;
+    debug_return_bool(true);
 }
 
-static void
-sudoers_policy_close(int exit_status, int error_code)
-{
-    if (sigsetjmp(error_jmp, 1)) {
-       /* called via error(), errorx() or log_error() */
-       return;
-    }
-
-    /* We do not currently log the exit status. */
-    if (error_code)
-       warningx("unable to execute %s: %s", safe_cmnd, strerror(error_code));
-
-    /* Close the session we opened in sudoers_policy_init_session(). */
-    if (ISSET(sudo_mode, MODE_RUN|MODE_EDIT))
-       (void)auth_end_session();
-
-    /* Free remaining references to password and group entries. */
-    pw_delref(sudo_user.pw);
-    pw_delref(runas_pw);
-    if (runas_gr != NULL)
-       gr_delref(runas_gr);
-}
-
-/*
- * The init_session function is called before executing the command
- * and before uid/gid changes occur.
- */
-static int
-sudoers_policy_init_session(struct passwd *pwd)
-{
-    if (sigsetjmp(error_jmp, 1)) {
-       /* called via error(), errorx() or log_error() */
-       return -1;
-    }
-
-    return auth_begin_session(pwd);
-}
-
-static int
+int
 sudoers_policy_main(int argc, char * const argv[], int pwflag, char *env_add[],
-    char **command_infop[], char **argv_out[], char **user_env_out[])
+    void *closure)
 {
-    static char *command_info[32]; /* XXX */
     char **edit_argv = NULL;
+    char *iolog_path = NULL;
+    mode_t cmnd_umask = 0777;
     struct sudo_nss *nss;
-    int cmnd_status = -1, validated;
-    volatile int info_len = 0;
-    volatile int rval = TRUE;
+    int cmnd_status = -1, oldlocale, validated;
+    volatile int rval = true;
+    debug_decl(sudoers_policy_main, SUDO_DEBUG_PLUGIN)
 
-    if (sigsetjmp(error_jmp, 1)) {
-       /* error recovery via error(), errorx() or log_error() */
+    /* XXX - would like to move this to policy.c but need the cleanup. */
+    if (fatal_setjmp() != 0) {
+       /* error recovery via fatal(), fatalx() or log_fatal() */
        rval = -1;
        goto done;
     }
 
     /* Is root even allowed to run sudo? */
     if (user_uid == 0 && !def_root_sudo) {
-        warningx("sudoers specifies that root is not allowed to sudo");
+        warningx(_("sudoers specifies that root is not allowed to sudo"));
         goto bad;
     }    
 
     /* Check for -C overriding def_closefrom. */
     if (user_closefrom >= 0 && user_closefrom != def_closefrom) {
        if (!def_closefrom_override) {
-           warningx("you are not permitted to use the -C option");
+           warningx(_("you are not permitted to use the -C option"));
            goto bad;
        }
        def_closefrom = user_closefrom;
@@ -335,38 +253,32 @@ sudoers_policy_main(int argc, char * const argv[], int pwflag, char *env_add[],
        NewArgv[0] = user_cmnd;
        NewArgv[1] = NULL;
     } else {
+       /* Must leave an extra slot before NewArgv for bash's --login */
        NewArgc = argc;
-       NewArgv = emalloc2(NewArgc + 1, sizeof(char *));
-       memcpy(NewArgv, argv, argc * sizeof(char *));
+       NewArgv = emalloc2(NewArgc + 2, sizeof(char *));
+       memcpy(++NewArgv, argv, argc * sizeof(char *));
        NewArgv[NewArgc] = NULL;
-       if (ISSET(sudo_mode, MODE_LOGIN_SHELL))
+       if (ISSET(sudo_mode, MODE_LOGIN_SHELL) && runas_pw != NULL)
            NewArgv[0] = estrdup(runas_pw->pw_shell);
     }
 
-    /* Find command in path */
-    cmnd_status = set_cmnd(sudo_mode);
-    if (cmnd_status == -1) {
-       rval = -1;
-       goto done;
-    }
+    /* If given the -P option, set the "preserve_groups" flag. */
+    if (ISSET(sudo_mode, MODE_PRESERVE_GROUPS))
+       def_preserve_groups = true;
 
-#ifdef HAVE_SETLOCALE
-    if (!setlocale(LC_ALL, def_sudoers_locale)) {
-       warningx("unable to set locale to \"%s\", using \"C\"",
-           def_sudoers_locale);
-       setlocale(LC_ALL, "C");
-    }
-#endif
+    /* Find command in path */
+    cmnd_status = set_cmnd();
 
     /*
-     * Check sudoers sources.
+     * Check sudoers sources, using the locale specified in sudoers.
      */
+    sudoers_setlocale(SUDOERS_LOCALE_SUDOERS, &oldlocale);
     validated = FLAG_NO_USER | FLAG_NO_HOST;
     tq_foreach_fwd(snl, nss) {
        validated = nss->lookup(nss, validated, pwflag);
 
        if (ISSET(validated, VALIDATE_OK)) {
-           /* Handle "= auth" in netsvc.conf */
+           /* Handle [SUCCESS=return] */
            if (nss->ret_if_found)
                break;
        } else {
@@ -376,13 +288,12 @@ sudoers_policy_main(int argc, char * const argv[], int pwflag, char *env_add[],
        }
     }
 
+    /* Restore user's locale. */
+    sudoers_setlocale(oldlocale, NULL);
+
     if (safe_cmnd == NULL)
        safe_cmnd = estrdup(user_cmnd);
 
-#ifdef HAVE_SETLOCALE
-    setlocale(LC_ALL, "");
-#endif
-
     /* If only a group was specified, set runas_pw based on invoking user. */
     if (runas_pw == NULL)
        set_runaspw(user_name);
@@ -397,17 +308,16 @@ sudoers_policy_main(int argc, char * const argv[], int pwflag, char *env_add[],
            pw = sudo_getpwuid(atoi(def_timestampowner + 1));
        else
            pw = sudo_getpwnam(def_timestampowner);
-       if (!pw)
-           log_error(0, "timestamp owner (%s): No such user",
+       if (pw != NULL) {
+           timestamp_uid = pw->pw_uid;
+           sudo_pw_delref(pw);
+       } else {
+           log_warning(0, N_("timestamp owner (%s): No such user"),
                def_timestampowner);
-       timestamp_uid = pw->pw_uid;
-       pw_delref(pw);
+           timestamp_uid = ROOT_UID;
+       }
     }
 
-    /* If given the -P option, set the "preserve_groups" flag. */
-    if (ISSET(sudo_mode, MODE_PRESERVE_GROUPS))
-       def_preserve_groups = TRUE;
-
     /* If no command line args and "shell_noargs" is not set, error out. */
     if (ISSET(sudo_mode, MODE_IMPLIED_SHELL) && !def_shell_noargs) {
        rval = -2; /* usage error */
@@ -415,14 +325,10 @@ sudoers_policy_main(int argc, char * const argv[], int pwflag, char *env_add[],
     }
 
     /* Bail if a tty is required and we don't have one.  */
-    if (def_requiretty) {
-       int fd = open(_PATH_TTY, O_RDWR|O_NOCTTY);
-       if (fd == -1) {
-           audit_failure(NewArgv, "no tty");
-           warningx("sorry, you must have a tty to run sudo");
-           goto bad;
-       } else
-           (void) close(fd);
+    if (def_requiretty && !tty_present()) {
+       audit_failure(NewArgv, N_("no tty"));
+       warningx(_("sorry, you must have a tty to run sudo"));
+       goto bad;
     }
 
     /*
@@ -431,18 +337,17 @@ sudoers_policy_main(int argc, char * const argv[], int pwflag, char *env_add[],
      */
     if (ISSET(sudo_mode, MODE_EDIT) ||
        (ISSET(sudo_mode, MODE_PRESERVE_ENV) && def_setenv))
-       def_env_reset = FALSE;
+       def_env_reset = false;
 
     /* Build a new environment that avoids any nasty bits. */
     rebuild_env();
 
     /* Require a password if sudoers says so.  */
-    if (def_authenticate) {
-       int rc = check_user(validated, sudo_mode);
-       if (rc != TRUE) {
-           rval = rc;
-           goto done;
-       }
+    rval = check_user(validated, sudo_mode);
+    if (rval != true) {
+       if (!ISSET(validated, VALIDATE_OK))
+           log_denial(validated, false);
+       goto done;
     }
 
     /* If run as root with SUDO_USER set, set sudo_user.pw to that user. */
@@ -453,41 +358,15 @@ sudoers_policy_main(int argc, char * const argv[], int pwflag, char *env_add[],
 
            if ((pw = sudo_getpwnam(prev_user)) != NULL) {
                    if (sudo_user.pw != NULL)
-                       pw_delref(sudo_user.pw);
+                       sudo_pw_delref(sudo_user.pw);
                    sudo_user.pw = pw;
-#ifdef HAVE_MBR_CHECK_MEMBERSHIP
-                   mbr_uid_to_uuid(user_uid, user_uuid);
-#endif
            }
        }
     }
 
     /* If the user was not allowed to run the command we are done. */
     if (!ISSET(validated, VALIDATE_OK)) {
-       if (ISSET(validated, FLAG_NO_USER | FLAG_NO_HOST)) {
-           audit_failure(NewArgv, "No user or host");
-           log_denial(validated, 1);
-       } else {
-           if (def_path_info) {
-               /*
-                * We'd like to not leak path info at all here, but that can
-                * *really* confuse the users.  To really close the leak we'd
-                * have to say "not allowed to run foo" even when the problem
-                * is just "no foo in path" since the user can trivially set
-                * their path to just contain a single dir.
-                */
-               log_denial(validated,
-                   !(cmnd_status == NOT_FOUND_DOT || cmnd_status == NOT_FOUND));
-               if (cmnd_status == NOT_FOUND)
-                   warningx("%s: command not found", user_cmnd);
-               else if (cmnd_status == NOT_FOUND_DOT)
-                   warningx("ignoring `%s' found in '.'\nUse `sudo ./%s' if this is the `%s' you wish to run.", user_cmnd, user_cmnd, user_cmnd);
-           } else {
-               /* Just tell the user they are not allowed to run foo. */
-               log_denial(validated, 1);
-           }
-           audit_failure(NewArgv, "validation failure");
-       }
+       log_failure(validated, cmnd_status);
        goto bad;
     }
 
@@ -496,41 +375,36 @@ sudoers_policy_main(int argc, char * const argv[], int pwflag, char *env_add[],
 
     /* Finally tell the user if the command did not exist. */
     if (cmnd_status == NOT_FOUND_DOT) {
-       audit_failure(NewArgv, "command in current directory");
-       warningx("ignoring `%s' found in '.'\nUse `sudo ./%s' if this is the `%s' you wish to run.", user_cmnd, user_cmnd, user_cmnd);
+       audit_failure(NewArgv, N_("command in current directory"));
+       warningx(_("ignoring `%s' found in '.'\nUse `sudo ./%s' if this is the `%s' you wish to run."), user_cmnd, user_cmnd, user_cmnd);
        goto bad;
     } else if (cmnd_status == NOT_FOUND) {
-       audit_failure(NewArgv, "%s: command not found", user_cmnd);
-       warningx("%s: command not found", user_cmnd);
+       if (ISSET(sudo_mode, MODE_CHECK)) {
+           audit_failure(NewArgv, N_("%s: command not found"), NewArgv[0]);
+           warningx(_("%s: command not found"), NewArgv[0]);
+       } else {
+           audit_failure(NewArgv, N_("%s: command not found"), user_cmnd);
+           warningx(_("%s: command not found"), user_cmnd);
+       }
        goto bad;
     }
 
     /* If user specified env vars make sure sudoers allows it. */
     if (ISSET(sudo_mode, MODE_RUN) && !def_setenv) {
        if (ISSET(sudo_mode, MODE_PRESERVE_ENV)) {
-           warningx("sorry, you are not allowed to preserve the environment");
+           warningx(_("sorry, you are not allowed to preserve the environment"));
            goto bad;
        } else
            validate_env_vars(sudo_user.env_vars);
     }
 
-    if (ISSET(sudo_mode, (MODE_RUN | MODE_EDIT)) && (def_log_input || def_log_output)) {
-       if (def_iolog_file && def_iolog_dir) {
-           command_info[info_len++] = expand_iolog_path("iolog_path=",
-               def_iolog_dir, def_iolog_file, &sudo_user.iolog_file);
+    if (ISSET(sudo_mode, (MODE_RUN | MODE_EDIT))) {
+       if ((def_log_input || def_log_output) && def_iolog_file && def_iolog_dir) {
+           const char prefix[] = "iolog_path=";
+           iolog_path = expand_iolog_path(prefix, def_iolog_dir,
+               def_iolog_file, &sudo_user.iolog_file);
            sudo_user.iolog_file++;
        }
-       if (def_log_input) {
-           command_info[info_len++] = estrdup("iolog_stdin=true");
-           command_info[info_len++] = estrdup("iolog_ttyin=true");
-       }
-       if (def_log_output) {
-           command_info[info_len++] = estrdup("iolog_stdout=true");
-           command_info[info_len++] = estrdup("iolog_stderr=true");
-           command_info[info_len++] = estrdup("iolog_ttyout=true");
-       }
-       if (def_compress_io)
-           command_info[info_len++] = estrdup("iolog_compress=true");
     }
 
     log_allowed(validated);
@@ -557,13 +431,9 @@ sudoers_policy_main(int argc, char * const argv[], int pwflag, char *env_add[],
      * unless umask_override is set.
      */
     if (def_umask != 0777) {
-       mode_t mask = def_umask;
-       if (!def_umask_override) {
-           mode_t omask = umask(mask);
-           mask |= omask;
-           umask(omask);
-       }
-       easprintf(&command_info[info_len++], "umask=0%o", (unsigned int)mask);
+       cmnd_umask = def_umask;
+       if (!def_umask_override)
+           cmnd_umask |= user_umask;
     }
 
     if (ISSET(sudo_mode, MODE_LOGIN_SHELL)) {
@@ -575,187 +445,87 @@ sudoers_policy_main(int argc, char * const argv[], int pwflag, char *env_add[],
        *p = '-';
        NewArgv[0] = p;
 
-       /* Set cwd to run user's homedir. */
-       command_info[info_len++] = fmt_string("cwd", runas_pw->pw_dir);
+       /*
+        * Newer versions of bash require the --login option to be used
+        * in conjunction with the -c option even if the shell name starts
+        * with a '-'.  Unfortunately, bash 1.x uses -login, not --login
+        * so this will cause an error for that.
+        */
+       if (NewArgc > 1 && strcmp(NewArgv[0], "-bash") == 0 &&
+           strcmp(NewArgv[1], "-c") == 0) {
+           /* Use the extra slot before NewArgv so we can store --login. */
+           NewArgv--;
+           NewArgc++;
+           NewArgv[0] = NewArgv[1];
+           NewArgv[1] = "--login";
+       }
 
-#if defined(__linux__) || defined(_AIX)
+#if defined(_AIX) || (defined(__linux__) && !defined(HAVE_PAM))
        /* Insert system-wide environment variables. */
-       read_env_file(_PATH_ENVIRONMENT, TRUE);
+       read_env_file(_PATH_ENVIRONMENT, true);
 #endif
+#ifdef HAVE_LOGIN_CAP_H
+       /* Set environment based on login class. */
+       if (login_class) {
+           login_cap_t *lc = login_getclass(login_class);
+           if (lc != NULL) {
+               setusercontext(lc, runas_pw, runas_pw->pw_uid, LOGIN_SETPATH|LOGIN_SETENV);
+               login_close(lc);
+           }
+       }
+#endif /* HAVE_LOGIN_CAP_H */
     }
 
     /* Insert system-wide environment variables. */
     if (def_env_file)
-       read_env_file(def_env_file, FALSE);
+       read_env_file(def_env_file, false);
 
     /* Insert user-specified environment variables. */
     insert_env_vars(sudo_user.env_vars);
 
-    /* Restore signal handlers before we exec. */
-    (void) sigaction(SIGINT, &saved_sa_int, NULL);
-    (void) sigaction(SIGQUIT, &saved_sa_quit, NULL);
-    (void) sigaction(SIGTSTP, &saved_sa_tstp, NULL);
-
     if (ISSET(sudo_mode, MODE_EDIT)) {
-       char *editor = find_editor(NewArgc - 1, NewArgv + 1, &edit_argv);
-       if (editor == NULL)
+       efree(safe_cmnd);
+       safe_cmnd = find_editor(NewArgc - 1, NewArgv + 1, &edit_argv);
+       if (safe_cmnd == NULL)
            goto bad;
-       command_info[info_len++] = fmt_string("command", editor);
-       command_info[info_len++] = estrdup("sudoedit=true");
-    } else {
-       command_info[info_len++] = fmt_string("command", safe_cmnd);
-    }
-    if (def_stay_setuid) {
-       easprintf(&command_info[info_len++], "runas_uid=%u",
-           (unsigned int)user_uid);
-       easprintf(&command_info[info_len++], "runas_gid=%u",
-           (unsigned int)user_gid);
-       easprintf(&command_info[info_len++], "runas_euid=%u",
-           (unsigned int)runas_pw->pw_uid);
-       easprintf(&command_info[info_len++], "runas_egid=%u",
-           runas_gr ? (unsigned int)runas_gr->gr_gid :
-           (unsigned int)runas_pw->pw_gid);
-    } else {
-       easprintf(&command_info[info_len++], "runas_uid=%u",
-           (unsigned int)runas_pw->pw_uid);
-       easprintf(&command_info[info_len++], "runas_gid=%u",
-           runas_gr ? (unsigned int)runas_gr->gr_gid :
-           (unsigned int)runas_pw->pw_gid);
-    }
-    if (def_preserve_groups) {
-       command_info[info_len++] = "preserve_groups=true";
-    } else if (runas_ngroups != -1) {
-       int i, len;
-       size_t glsize;
-       char *cp, *gid_list;
-
-       glsize = sizeof("runas_groups=") - 1 + (runas_ngroups * (MAX_UID_T_LEN + 1));
-       gid_list = emalloc(glsize);
-       memcpy(gid_list, "runas_groups=", sizeof("runas_groups=") - 1);
-       cp = gid_list + sizeof("runas_groups=") - 1;
-       for (i = 0; i < runas_ngroups; i++) {
-           /* XXX - check rval */
-           len = snprintf(cp, glsize - (cp - gid_list), "%s%u",
-                i ? "," : "", (unsigned int) runas_groups[i]);
-           cp += len;
-       }
-       command_info[info_len++] = gid_list;
     }
-    if (def_closefrom >= 0)
-       easprintf(&command_info[info_len++], "closefrom=%d", def_closefrom);
-    if (def_noexec)
-       command_info[info_len++] = estrdup("noexec=true");
-    if (def_noexec_file)
-       command_info[info_len++] = fmt_string("noexec_file", def_noexec_file);
-    if (def_set_utmp)
-       command_info[info_len++] = estrdup("set_utmp=true");
-    if (def_utmp_runas)
-       command_info[info_len++] = fmt_string("utmp_user", runas_pw->pw_name);
-#ifdef HAVE_LOGIN_CAP_H
-    if (lc != NULL)
-       command_info[info_len++] = fmt_string("login_class", lc->lc_class);
-#endif /* HAVE_LOGIN_CAP_H */
-#ifdef HAVE_SELINUX
-    if (user_role != NULL)
-       command_info[info_len++] = fmt_string("selinux_role", user_role);
-    if (user_type != NULL)
-       command_info[info_len++] = fmt_string("selinux_type", user_type);
-#endif /* HAVE_SELINUX */
 
     /* Must audit before uid change. */
     audit_success(NewArgv);
 
-    *command_infop = command_info;
+    /* Setup execution environment to pass back to front-end. */
+    rval = sudoers_policy_exec_setup(edit_argv ? edit_argv : NewArgv,
+       env_get(), cmnd_umask, iolog_path, closure);
 
-    *argv_out = edit_argv ? edit_argv : NewArgv;
-    *user_env_out = env_get(); /* our private copy */
+    /* Zero out stashed copy of environment, it is owned by the front-end. */
+    env_init(NULL);
 
     goto done;
 
 bad:
-    rval = FALSE;
+    rval = false;
 
 done:
+    fatal_disable_setjmp();
     rewind_perms();
 
     /* Close the password and group files and free up memory. */
     sudo_endpwent();
     sudo_endgrent();
 
-    return rval;
-}
-
-static int
-sudoers_policy_check(int argc, char * const argv[], char *env_add[],
-    char **command_infop[], char **argv_out[], char **user_env_out[])
-{
-    if (!ISSET(sudo_mode, MODE_EDIT))
-       SET(sudo_mode, MODE_RUN);
-
-    return sudoers_policy_main(argc, argv, 0, env_add, command_infop,
-       argv_out, user_env_out);
-}
-
-static int
-sudoers_policy_validate(void)
-{
-    user_cmnd = "validate";
-    SET(sudo_mode, MODE_VALIDATE);
-
-    return sudoers_policy_main(0, NULL, I_VERIFYPW, NULL, NULL, NULL, NULL);
-}
-
-static void
-sudoers_policy_invalidate(int remove)
-{
-    user_cmnd = "kill";
-    if (sigsetjmp(error_jmp, 1) == 0) {
-       remove_timestamp(remove);
-       plugin_cleanup(0);
-    }
-}
-
-static int
-sudoers_policy_list(int argc, char * const argv[], int verbose,
-    const char *list_user)
-{
-    int rval;
-
-    user_cmnd = "list";
-    if (argc)
-       SET(sudo_mode, MODE_CHECK);
-    else
-       SET(sudo_mode, MODE_LIST);
-    if (verbose)
-       long_list = 1;
-    if (list_user) {
-       list_pw = sudo_getpwnam(list_user);
-       if (list_pw == NULL) {
-           warningx("unknown user: %s", list_user);
-           return -1;
-       }
-    }
-    rval = sudoers_policy_main(argc, argv, I_LISTPW, NULL, NULL, NULL, NULL);
-    if (list_user) {
-       pw_delref(list_pw);
-       list_pw = NULL;
-    }
-
-    return rval;
+    debug_return_bool(rval);
 }
 
 /*
- * Initialize timezone, set umask, fill in ``sudo_user'' struct and
- * load the ``interfaces'' array.
+ * Initialize timezone and fill in ``sudo_user'' struct.
  */
 static void
 init_vars(char * const envp[])
 {
     char * const * ep;
+    debug_decl(init_vars, SUDO_DEBUG_PLUGIN)
 
-#ifdef HAVE_TZSET
-    (void) tzset();            /* set the timezone if applicable */
-#endif /* HAVE_TZSET */
+    sudoers_initlocale(setlocale(LC_ALL, NULL), def_sudoers_locale);
 
     for (ep = envp; *ep; ep++) {
        /* XXX - don't fill in if empty string */
@@ -778,34 +548,43 @@ init_vars(char * const envp[])
     }
 
     /*
-     * Get a local copy of the user's struct passwd with the shadow password
-     * if necessary.  It is assumed that euid is 0 at this point so we
-     * can read the shadow passwd file if necessary.
+     * Get a local copy of the user's struct passwd if we don't already
+     * have one.
      */
-    if ((sudo_user.pw = sudo_getpwnam(user_name)) == NULL) {
-       struct passwd pw;
+    if (sudo_user.pw == NULL) {
+       if ((sudo_user.pw = sudo_getpwnam(user_name)) == NULL) {
+           /*
+            * It is not unusual for users to place "sudo -k" in a .logout
+            * file which can cause sudo to be run during reboot after the
+            * YP/NIS/NIS+/LDAP/etc daemon has died.
+            */
+           if (sudo_mode == MODE_KILL || sudo_mode == MODE_INVALIDATE)
+               fatalx(_("unknown uid: %u"), (unsigned int) user_uid);
+
+           /* Need to make a fake struct passwd for the call to log_fatal(). */
+           sudo_user.pw = sudo_mkpwent(user_name, user_uid, user_gid, NULL, NULL);
+           log_fatal(0, N_("unknown uid: %u"), (unsigned int) user_uid);
+           /* NOTREACHED */
+       }
+    }
 
-       /* Create a fake struct passwd for log_error(). */
-       memset(&pw, 0, sizeof(pw));
-       pw.pw_uid = getuid();
-       pw.pw_name = user_name;
-       sudo_user.pw = &pw;
+    /*
+     * Get group list.
+     */
+    if (user_group_list == NULL)
+       user_group_list = sudo_get_grlist(sudo_user.pw);
 
-       /*
-        * It is not unusual for users to place "sudo -k" in a .logout
-        * file which can cause sudo to be run during reboot after the
-        * YP/NIS/NIS+/LDAP/etc daemon has died.
-        */
-       if (sudo_mode == MODE_KILL || sudo_mode == MODE_INVALIDATE)
-           errorx(1, "unknown user: %s", user_name);
-       log_error(0, "unknown user: %s", user_name);
-       /* NOTREACHED */
-    }
-#ifdef HAVE_MBR_CHECK_MEMBERSHIP
-    mbr_uid_to_uuid(user_uid, user_uuid);
-#endif
+    /* Set runas callback. */
+    sudo_defs_table[I_RUNAS_DEFAULT].callback = cb_runas_default;
+
+    /* Set locale callback. */
+    sudo_defs_table[I_SUDOERS_LOCALE].callback = cb_sudoers_locale;
 
-    /* It is now safe to use log_error() and set_perms() */
+    /* Set maxseq callback. */
+    sudo_defs_table[I_MAXSEQ].callback = io_set_max_sessid;
+
+    /* It is now safe to use log_fatal() and set_perms() */
+    debug_return;
 }
 
 /*
@@ -813,14 +592,15 @@ init_vars(char * const envp[])
  * and apply any command-specific defaults entries.
  */
 static int
-set_cmnd(int sudo_mode)
+set_cmnd(void)
 {
     int rval;
     char *path = user_path;
+    debug_decl(set_cmnd, SUDO_DEBUG_PLUGIN)
 
     /* Resolve the path and return. */
     rval = FOUND;
-    user_stat = emalloc(sizeof(struct stat));
+    user_stat = ecalloc(1, sizeof(struct stat));
 
     /* Default value for cmnd, overridden below. */
     if (user_cmnd == NULL)
@@ -845,25 +625,44 @@ set_cmnd(int sudo_mode)
 
        /* set user_args */
        if (NewArgc > 1) {
-           char *to, **from;
+           char *to, *from, **av;
            size_t size, n;
 
            /* Alloc and build up user_args. */
-           for (size = 0, from = NewArgv + 1; *from; from++)
-               size += strlen(*from) + 1;
+           for (size = 0, av = NewArgv + 1; *av; av++)
+               size += strlen(*av) + 1;
            user_args = emalloc(size);
-           for (to = user_args, from = NewArgv + 1; *from; from++) {
-               n = strlcpy(to, *from, size - (to - user_args));
-               if (n >= size - (to - user_args))
-                   errorx(1, "internal error, set_cmnd() overflow");
-               to += n;
-               *to++ = ' ';
+           if (ISSET(sudo_mode, MODE_SHELL|MODE_LOGIN_SHELL)) {
+               /*
+                * When running a command via a shell, the sudo front-end
+                * escapes potential meta chars.  We unescape non-spaces
+                * for sudoers matching and logging purposes.
+                */
+               for (to = user_args, av = NewArgv + 1; (from = *av); av++) {
+                   while (*from) {
+                       if (from[0] == '\\' && !isspace((unsigned char)from[1]))
+                           from++;
+                       *to++ = *from++;
+                   }
+                   *to++ = ' ';
+               }
+               *--to = '\0';
+           } else {
+               for (to = user_args, av = NewArgv + 1; *av; av++) {
+                   n = strlcpy(to, *av, size - (to - user_args));
+                   if (n >= size - (to - user_args))
+                       fatalx(_("internal error, %s overflow"), "set_cmnd()");
+                   to += n;
+                   *to++ = ' ';
+               }
+               *--to = '\0';
            }
-           *--to = '\0';
        }
     }
-    if (strlen(user_cmnd) >= PATH_MAX)
-       errorx(1, "%s: file name too long", user_cmnd);
+    if (strlen(user_cmnd) >= PATH_MAX) {
+       errno = ENAMETOOLONG;
+       fatal("%s", user_cmnd);
+    }
 
     if ((user_base = strrchr(user_cmnd, '/')) != NULL)
        user_base++;
@@ -871,12 +670,9 @@ set_cmnd(int sudo_mode)
        user_base = user_cmnd;
 
     if (!update_defaults(SETDEF_CMND))
-       log_error(NO_STDERR|NO_EXIT, "problem with defaults entries");
-
-    if (!runas_user && !runas_group)
-       set_runaspw(def_runas_default); /* may have been updated above */
+       log_warning(NO_STDERR, N_("problem with defaults entries"));
 
-    return rval;
+    debug_return_int(rval);
 }
 
 /*
@@ -884,98 +680,87 @@ set_cmnd(int sudo_mode)
  * Returns a handle to the sudoers file or NULL on error.
  */
 FILE *
-open_sudoers(const char *sudoers, int doedit, int *keepopen)
+open_sudoers(const char *sudoers, bool doedit, bool *keepopen)
 {
-    struct stat statbuf;
+    struct stat sb;
     FILE *fp = NULL;
-    int rootstat;
-
-    /*
-     * Fix the mode and group on sudoers file from old default.
-     * Only works if file system is readable/writable by root.
-     */
-    if ((rootstat = stat_sudoers(sudoers, &statbuf)) == 0 &&
-       sudoers_uid == statbuf.st_uid && sudoers_mode != 0400 &&
-       (statbuf.st_mode & 0007777) == 0400) {
-
-       if (chmod(sudoers, sudoers_mode) == 0) {
-           warningx("fixed mode on %s", sudoers);
-           SET(statbuf.st_mode, sudoers_mode);
-           if (statbuf.st_gid != sudoers_gid) {
-               if (chown(sudoers, (uid_t) -1, sudoers_gid) == 0) {
-                   warningx("set group on %s", sudoers);
-                   statbuf.st_gid = sudoers_gid;
-               } else
-                   warning("unable to set group on %s", sudoers);
-           }
-       } else
-           warning("unable to fix mode on %s", sudoers);
-    }
+    debug_decl(open_sudoers, SUDO_DEBUG_PLUGIN)
 
-    /*
-     * Sanity checks on sudoers file.  Must be done as sudoers
-     * file owner.  We already did a stat as root, so use that
-     * data if we can't stat as sudoers file owner.
-     */
     set_perms(PERM_SUDOERS);
 
-    if (rootstat != 0 && stat_sudoers(sudoers, &statbuf) != 0)
-       log_error(USE_ERRNO|NO_EXIT, "can't stat %s", sudoers);
-    else if (!S_ISREG(statbuf.st_mode))
-       log_error(NO_EXIT, "%s is not a regular file", sudoers);
-    else if ((statbuf.st_mode & 07577) != sudoers_mode)
-       log_error(NO_EXIT, "%s is mode 0%o, should be 0%o", sudoers,
-           (unsigned int) (statbuf.st_mode & 07777),
-           (unsigned int) sudoers_mode);
-    else if (statbuf.st_uid != sudoers_uid)
-       log_error(NO_EXIT, "%s is owned by uid %u, should be %u", sudoers,
-           (unsigned int) statbuf.st_uid, (unsigned int) sudoers_uid);
-    else if (statbuf.st_gid != sudoers_gid)
-       log_error(NO_EXIT, "%s is owned by gid %u, should be %u", sudoers,
-           (unsigned int) statbuf.st_gid, (unsigned int) sudoers_gid);
-    else if ((fp = fopen(sudoers, "r")) == NULL)
-       log_error(USE_ERRNO|NO_EXIT, "can't open %s", sudoers);
-    else {
-       /*
-        * Make sure we can actually read sudoers so we can present the
-        * user with a reasonable error message (unlike the lexer).
-        */
-       if (statbuf.st_size != 0 && fgetc(fp) == EOF) {
-           log_error(USE_ERRNO|NO_EXIT, "can't read %s", sudoers);
-           fclose(fp);
-           fp = NULL;
-       }
-    }
-
-    if (fp != NULL) {
-       rewind(fp);
-       (void) fcntl(fileno(fp), F_SETFD, 1);
+    switch (sudo_secure_file(sudoers, sudoers_uid, sudoers_gid, &sb)) {
+       case SUDO_PATH_SECURE:
+           /*
+            * If we are expecting sudoers to be group readable but
+            * it is not, we must open the file as root, not uid 1.
+            */
+           if (sudoers_uid == ROOT_UID && (sudoers_mode & S_IRGRP)) {
+               if ((sb.st_mode & S_IRGRP) == 0) {
+                   restore_perms();
+                   set_perms(PERM_ROOT);
+               }
+           }
+           /*
+            * Open sudoers and make sure we can read it so we can present
+            * the user with a reasonable error message (unlike the lexer).
+            */
+           if ((fp = fopen(sudoers, "r")) == NULL) {
+               log_warning(USE_ERRNO, N_("unable to open %s"), sudoers);
+           } else {
+               if (sb.st_size != 0 && fgetc(fp) == EOF) {
+                   log_warning(USE_ERRNO, N_("unable to read %s"),
+                       sudoers);
+                   fclose(fp);
+                   fp = NULL;
+               } else {
+                   /* Rewind fp and set close on exec flag. */
+                   rewind(fp);
+                   (void) fcntl(fileno(fp), F_SETFD, 1);
+               }
+           }
+           break;
+       case SUDO_PATH_MISSING:
+           log_warning(USE_ERRNO, N_("unable to stat %s"), sudoers);
+           break;
+       case SUDO_PATH_BAD_TYPE:
+           log_warning(0, N_("%s is not a regular file"), sudoers);
+           break;
+       case SUDO_PATH_WRONG_OWNER:
+           log_warning(0, N_("%s is owned by uid %u, should be %u"),
+               sudoers, (unsigned int) sb.st_uid, (unsigned int) sudoers_uid);
+           break;
+       case SUDO_PATH_WORLD_WRITABLE:
+           log_warning(0, N_("%s is world writable"), sudoers);
+           break;
+       case SUDO_PATH_GROUP_WRITABLE:
+           log_warning(0, N_("%s is owned by gid %u, should be %u"),
+               sudoers, (unsigned int) sb.st_gid, (unsigned int) sudoers_gid);
+           break;
+       default:
+           /* NOTREACHED */
+           break;
     }
 
     restore_perms();           /* change back to root */
-    return fp;
+
+    debug_return_ptr(fp);
 }
 
 #ifdef HAVE_LOGIN_CAP_H
 static void
 set_loginclass(struct passwd *pw)
 {
-    int errflags;
+    const int errflags = NO_MAIL|MSG_ONLY;
+    login_cap_t *lc;
+    debug_decl(set_loginclass, SUDO_DEBUG_PLUGIN)
 
-    /*
-     * Don't make it a fatal error if the user didn't specify the login
-     * class themselves.  We do this because if login.conf gets
-     * corrupted we want the admin to be able to use sudo to fix it.
-     */
-    if (login_class)
-       errflags = NO_MAIL|MSG_ONLY;
-    else
-       errflags = NO_MAIL|MSG_ONLY|NO_EXIT;
+    if (!def_use_loginclass)
+       debug_return;
 
     if (login_class && strcmp(login_class, "-") != 0) {
        if (user_uid != 0 &&
            strcmp(runas_user ? runas_user : def_runas_default, "root") != 0)
-           errorx(1, "only root can use -c %s", login_class);
+           fatalx(_("only root can use `-c %s'"), login_class);
     } else {
        login_class = pw->pw_class;
        if (!login_class || !*login_class)
@@ -983,12 +768,22 @@ set_loginclass(struct passwd *pw)
                (pw->pw_uid == 0) ? LOGIN_DEFROOTCLASS : LOGIN_DEFCLASS;
     }
 
+    /* Make sure specified login class is valid. */
     lc = login_getclass(login_class);
     if (!lc || !lc->lc_class || strcmp(lc->lc_class, login_class) != 0) {
-       log_error(errflags, "unknown login class: %s", login_class);
-       if (!lc)
-           lc = login_getclass(NULL);  /* needed for login_getstyle() later */
+       /*
+        * Don't make it a fatal error if the user didn't specify the login
+        * class themselves.  We do this because if login.conf gets
+        * corrupted we want the admin to be able to use sudo to fix it.
+        */
+       if (login_class)
+           log_fatal(errflags, N_("unknown login class: %s"), login_class);
+       else
+           log_warning(errflags, N_("unknown login class: %s"), login_class);
+       def_use_loginclass = false;
     }
+    login_close(lc);
+    debug_return;
 }
 #else
 static void
@@ -997,44 +792,38 @@ set_loginclass(struct passwd *pw)
 }
 #endif /* HAVE_LOGIN_CAP_H */
 
+#ifndef AI_FQDN
+# define AI_FQDN AI_CANONNAME
+#endif
+
 /*
  * Look up the fully qualified domain name and set user_host and user_shost.
+ * Use AI_FQDN if available since "canonical" is not always the same as fqdn.
  */
-void
+static void
 set_fqdn(void)
 {
-#ifdef HAVE_GETADDRINFO
     struct addrinfo *res0, hint;
-#else
-    struct hostent *hp;
-#endif
     char *p;
+    debug_decl(set_fqdn, SUDO_DEBUG_PLUGIN)
 
-#ifdef HAVE_GETADDRINFO
     zero_bytes(&hint, sizeof(hint));
     hint.ai_family = PF_UNSPEC;
-    hint.ai_flags = AI_CANONNAME;
+    hint.ai_flags = AI_FQDN;
     if (getaddrinfo(user_host, NULL, &hint, &res0) != 0) {
-#else
-    if (!(hp = gethostbyname(user_host))) {
-#endif
-       log_error(MSG_ONLY|NO_EXIT,
-           "unable to resolve host %s", user_host);
+       log_warning(MSG_ONLY, N_("unable to resolve host %s"), user_host);
     } else {
        if (user_shost != user_host)
            efree(user_shost);
        efree(user_host);
-#ifdef HAVE_GETADDRINFO
        user_host = estrdup(res0->ai_canonname);
        freeaddrinfo(res0);
-#else
-       user_host = estrdup(hp->h_name);
-#endif
+       if ((p = strchr(user_host, '.')) != NULL)
+           user_shost = estrndup(user_host, (size_t)(p - user_host));
+       else
+           user_shost = user_host;
     }
-    if ((p = strchr(user_host, '.')) != NULL)
-       user_shost = estrndup(user_host, (size_t)(p - user_host));
-    else
-       user_shost = user_host;
+    debug_return;
 }
 
 /*
@@ -1042,19 +831,20 @@ set_fqdn(void)
  * and store it in runas_pw.  By default, commands run as "root".
  */
 static void
-set_runaspw(char *user)
+set_runaspw(const char *user)
 {
+    debug_decl(set_runaspw, SUDO_DEBUG_PLUGIN)
+
     if (runas_pw != NULL)
-       pw_delref(runas_pw);
+       sudo_pw_delref(runas_pw);
     if (*user == '#') {
        if ((runas_pw = sudo_getpwuid(atoi(user + 1))) == NULL)
            runas_pw = sudo_fakepwnam(user, runas_gr ? runas_gr->gr_gid : 0);
     } else {
-       if ((runas_pw = sudo_getpwnam(user)) == NULL) {
-           audit_failure(NewArgv, "unknown user: %s", user);
-           log_error(NO_MAIL|MSG_ONLY, "unknown user: %s", user);
-       }
+       if ((runas_pw = sudo_getpwnam(user)) == NULL)
+           log_fatal(NO_MAIL|MSG_ONLY, N_("unknown user: %s"), user);
     }
+    debug_return;
 }
 
 /*
@@ -1062,280 +852,77 @@ set_runaspw(char *user)
  * and store it in runas_gr.
  */
 static void
-set_runasgr(char *group)
+set_runasgr(const char *group)
 {
+    debug_decl(set_runasgr, SUDO_DEBUG_PLUGIN)
+
     if (runas_gr != NULL)
-       gr_delref(runas_gr);
+       sudo_gr_delref(runas_gr);
     if (*group == '#') {
        if ((runas_gr = sudo_getgrgid(atoi(group + 1))) == NULL)
            runas_gr = sudo_fakegrnam(group);
     } else {
        if ((runas_gr = sudo_getgrnam(group)) == NULL)
-           log_error(NO_MAIL|MSG_ONLY, "unknown group: %s", group);
+           log_fatal(NO_MAIL|MSG_ONLY, N_("unknown group: %s"), group);
     }
+    debug_return;
 }
 
 /*
- * Cleanup hook for error()/errorx()
+ * Callback for runas_default sudoers setting.
  */
-void
-plugin_cleanup(int gotsignal)
+static int
+cb_runas_default(const char *user)
 {
-    struct sudo_nss *nss;
-
-    if (!gotsignal) {
-       if (snl != NULL) {
-           tq_foreach_fwd(snl, nss)
-               nss->close(nss);
-       }
-       if (def_group_plugin)
-           group_plugin_unload();
-       sudo_endpwent();
-       sudo_endgrent();
-    }
+    /* Only reset runaspw if user didn't specify one. */
+    if (!runas_user && !runas_group)
+       set_runaspw(user);
+    return true;
 }
 
+/*
+ * Callback for sudoers_locale sudoers setting.
+ */
 static int
-sudoers_policy_version(int verbose)
+cb_sudoers_locale(const char *locale)
 {
-    if (sigsetjmp(error_jmp, 1)) {
-       /* error recovery via error(), errorx() or log_error() */
-       return -1;
-    }
-
-    sudo_printf(SUDO_CONV_INFO_MSG, "Sudoers policy plugin version %s\n",
-       PACKAGE_VERSION);
-    sudo_printf(SUDO_CONV_INFO_MSG, "Sudoers file grammar version %d\n",
-       SUDOERS_GRAMMAR_VERSION);
-
-    if (verbose) {
-       sudo_printf(SUDO_CONV_INFO_MSG, "\nSudoers path: %s\n", sudoers_file);
-#ifdef HAVE_LDAP
-# ifdef _PATH_NSSWITCH_CONF
-       sudo_printf(SUDO_CONV_INFO_MSG, "nsswitch path: %s\n", _PATH_NSSWITCH_CONF);
-# endif
-       sudo_printf(SUDO_CONV_INFO_MSG, "ldap.conf path: %s\n", _PATH_LDAP_CONF);
-       sudo_printf(SUDO_CONV_INFO_MSG, "ldap.secret path: %s\n", _PATH_LDAP_SECRET);
-#endif
-       dump_auth_methods();
-       dump_defaults();
-       sudo_printf(SUDO_CONV_INFO_MSG, "\n");
-       dump_interfaces(interfaces_string);
-       sudo_printf(SUDO_CONV_INFO_MSG, "\n");
-    }
-    return TRUE;
+    sudoers_initlocale(NULL, locale);
+    return true;
 }
 
-static int
-deserialize_info(char * const settings[], char * const user_info[])
+/*
+ * Cleanup hook for fatal()/fatalx()
+ */
+void
+sudoers_cleanup(void)
 {
-    char * const *cur;
-    const char *p;
-    int flags = 0;
-
-#define MATCHES(s, v) (strncmp(s, v, sizeof(v) - 1) == 0)
-
-    /* Parse command line settings. */
-    user_closefrom = -1;
-    for (cur = settings; *cur != NULL; cur++) {
-       if (MATCHES(*cur, "closefrom=")) {
-           user_closefrom = atoi(*cur + sizeof("closefrom=") - 1);
-           continue;
-       }
-       if (MATCHES(*cur, "debug_level=")) {
-           debug_level = atoi(*cur + sizeof("debug_level=") - 1);
-           continue;
-       }
-       if (MATCHES(*cur, "runas_user=")) {
-           runas_user = *cur + sizeof("runas_user=") - 1;
-           continue;
-       }
-       if (MATCHES(*cur, "runas_group=")) {
-           runas_group = *cur + sizeof("runas_group=") - 1;
-           continue;
-       }
-       if (MATCHES(*cur, "prompt=")) {
-           user_prompt = *cur + sizeof("prompt=") - 1;
-           def_passprompt_override = TRUE;
-           continue;
-       }
-       if (MATCHES(*cur, "set_home=")) {
-           if (atobool(*cur + sizeof("set_home=") - 1) == TRUE)
-               SET(flags, MODE_RESET_HOME);
-           continue;
-       }
-       if (MATCHES(*cur, "preserve_environment=")) {
-           if (atobool(*cur + sizeof("preserve_environment=") - 1) == TRUE)
-               SET(flags, MODE_PRESERVE_ENV);
-           continue;
-       }
-       if (MATCHES(*cur, "run_shell=")) {
-           if (atobool(*cur + sizeof("run_shell=") - 1) == TRUE)
-               SET(flags, MODE_SHELL);
-           continue;
-       }
-       if (MATCHES(*cur, "login_shell=")) {
-           if (atobool(*cur + sizeof("login_shell=") - 1) == TRUE) {
-               SET(flags, MODE_LOGIN_SHELL);
-               def_env_reset = TRUE;
-           }
-           continue;
-       }
-       if (MATCHES(*cur, "implied_shell=")) {
-           if (atobool(*cur + sizeof("implied_shell=") - 1) == TRUE)
-               SET(flags, MODE_IMPLIED_SHELL);
-           continue;
-       }
-       if (MATCHES(*cur, "preserve_groups=")) {
-           if (atobool(*cur + sizeof("preserve_groups=") - 1) == TRUE)
-               SET(flags, MODE_PRESERVE_GROUPS);
-           continue;
-       }
-       if (MATCHES(*cur, "ignore_ticket=")) {
-           if (atobool(*cur + sizeof("ignore_ticket=") - 1) == TRUE)
-               SET(flags, MODE_IGNORE_TICKET);
-           continue;
-       }
-       if (MATCHES(*cur, "noninteractive=")) {
-           if (atobool(*cur + sizeof("noninteractive=") - 1) == TRUE)
-               SET(flags, MODE_NONINTERACTIVE);
-           continue;
-       }
-       if (MATCHES(*cur, "sudoedit=")) {
-           if (atobool(*cur + sizeof("sudoedit=") - 1) == TRUE)
-               SET(flags, MODE_EDIT);
-           continue;
-       }
-       if (MATCHES(*cur, "login_class=")) {
-           login_class = *cur + sizeof("login_class=") - 1;
-           def_use_loginclass = TRUE;
-           continue;
-       }
-#ifdef HAVE_SELINUX
-       if (MATCHES(*cur, "selinux_role=")) {
-           user_role = *cur + sizeof("selinux_role=") - 1;
-           continue;
-       }
-       if (MATCHES(*cur, "selinux_type=")) {
-           user_type = *cur + sizeof("selinux_type=") - 1;
-           continue;
-       }
-#endif /* HAVE_SELINUX */
-#ifdef HAVE_BSD_AUTH_H
-       if (MATCHES(*cur, "bsdauth_type=")) {
-           login_style = *cur + sizeof("bsdauth_type=") - 1;
-           continue;
-       }
-#endif /* HAVE_BSD_AUTH_H */
-#if !defined(HAVE_GETPROGNAME) && !defined(HAVE___PROGNAME)
-       if (MATCHES(*cur, "progname=")) {
-           setprogname(*cur + sizeof("progname=") - 1);
-           continue;
-       }
-#endif
-       if (MATCHES(*cur, "network_addrs=")) {
-           interfaces_string = *cur + sizeof("network_addrs=") - 1;
-           set_interfaces(interfaces_string);
-           continue;
-       }
-       if (MATCHES(*cur, "sudoers_file=")) {
-           sudoers_file = *cur + sizeof("sudoers_file=") - 1;
-           continue;
-       }
-       if (MATCHES(*cur, "sudoers_uid=")) {
-           sudoers_uid = (uid_t) atoi(*cur + sizeof("sudoers_uid=") - 1);
-           continue;
-       }
-       if (MATCHES(*cur, "sudoers_gid=")) {
-           sudoers_gid = (gid_t) atoi(*cur + sizeof("sudoers_gid=") - 1);
-           continue;
-       }
-       if (MATCHES(*cur, "sudoers_mode=")) {
-           sudoers_mode = (mode_t) strtol(*cur + sizeof("sudoers_mode=") - 1,
-               NULL, 8);
-           continue;
-       }
-    }
-
-    for (cur = user_info; *cur != NULL; cur++) {
-       if (MATCHES(*cur, "user=")) {
-           user_name = estrdup(*cur + sizeof("user=") - 1);
-           continue;
-       }
-       if (MATCHES(*cur, "uid=")) {
-           user_uid = (uid_t) atoi(*cur + sizeof("uid=") - 1);
-           continue;
-       }
-       if (MATCHES(*cur, "gid=")) {
-           user_gid = (gid_t) atoi(*cur + sizeof("gid=") - 1);
-           continue;
-       }
-       if (MATCHES(*cur, "groups=")) {
-           /* Count number of groups */
-           const char *val = *cur + sizeof("groups=") - 1;
-           const char *cp;
-           if (val[0] != '\0') {
-               user_ngroups = 1;
-               for (cp = val; *cp != '\0'; cp++) {
-                   if (*cp == ',')
-                       user_ngroups++;
-               }
+    struct sudo_nss *nss;
+    debug_decl(sudoers_cleanup, SUDO_DEBUG_PLUGIN)
 
-               user_groups = emalloc2(user_ngroups, sizeof(GETGROUPS_T));
-               user_ngroups = 0;
-               cp = val;
-               for (;;) {
-                   /* XXX - strtol would be better here */
-                   user_groups[user_ngroups++] = atoi(cp);
-                   cp = strchr(cp, ',');
-                   if (cp == NULL)
-                       break;
-                   cp++; /* skip over comma */
-               }
-           }
-           continue;
-       }
-       if (MATCHES(*cur, "cwd=")) {
-           user_cwd = estrdup(*cur + sizeof("cwd=") - 1);
-           continue;
-       }
-       if (MATCHES(*cur, "tty=")) {
-           user_tty = user_ttypath = estrdup(*cur + sizeof("tty=") - 1);
-           if (strncmp(user_tty, _PATH_DEV, sizeof(_PATH_DEV) - 1) == 0)
-               user_tty += sizeof(_PATH_DEV) - 1;
-           continue;
-       }
-       if (MATCHES(*cur, "host=")) {
-           user_host = user_shost = estrdup(*cur + sizeof("host=") - 1);
-           if ((p = strchr(user_host, '.')))
-               user_shost = estrndup(user_host, (size_t)(p - user_host));
-           continue;
-       }
-       if (MATCHES(*cur, "lines=")) {
-           sudo_user.lines = atoi(*cur + sizeof("lines=") - 1);
-           continue;
-       }
-       if (MATCHES(*cur, "cols=")) {
-           sudo_user.cols = atoi(*cur + sizeof("cols=") - 1);
-           continue;
-       }
+    if (snl != NULL) {
+       tq_foreach_fwd(snl, nss)
+           nss->close(nss);
     }
-    if (user_cwd == NULL)
-       user_cwd = "unknown";
-    if (user_tty == NULL)
-       user_tty = "unknown"; /* user_ttypath remains NULL */
+    if (def_group_plugin)
+       group_plugin_unload();
+    sudo_endpwent();
+    sudo_endgrent();
 
-#undef MATCHES
-    return flags;
+    debug_return;
 }
 
 static char *
-resolve_editor(char *editor, int nfiles, char **files, char ***argv_out)
+resolve_editor(const char *ed, size_t edlen, int nfiles, char **files, char ***argv_out)
 {
-    char *cp, **nargv, *editor_path = NULL;
-    int ac, i, nargc, wasblank;
+    char *cp, **nargv, *editor, *editor_path = NULL;
+    int ac, i, nargc;
+    bool wasblank;
+    debug_decl(resolve_editor, SUDO_DEBUG_PLUGIN)
 
-    editor = estrdup(editor); /* becomes part of argv_out */
+    /* Note: editor becomes part of argv_out and is not freed. */
+    editor = emalloc(edlen + 1);
+    memcpy(editor, ed, edlen);
+    editor[edlen] = '\0';
 
     /*
      * Split editor into an argument vector; editor is reused (do not free).
@@ -1343,11 +930,11 @@ resolve_editor(char *editor, int nfiles, char **files, char ***argv_out)
      * line args so look for those and alloc space for them too.
      */
     nargc = 1;
-    for (wasblank = FALSE, cp = editor; *cp != '\0'; cp++) {
+    for (wasblank = false, cp = editor; *cp != '\0'; cp++) {
        if (isblank((unsigned char) *cp))
-           wasblank = TRUE;
+           wasblank = true;
        else if (wasblank) {
-           wasblank = FALSE;
+           wasblank = false;
            nargc++;
        }
     }
@@ -1356,7 +943,7 @@ resolve_editor(char *editor, int nfiles, char **files, char ***argv_out)
     if (cp == NULL ||
        find_path(cp, &editor_path, NULL, getenv("PATH"), 0) != FOUND) {
        efree(editor);
-       return NULL;
+       debug_return_str(NULL);
     }
     nargv = (char **) emalloc2(nargc + 1 + nfiles + 1, sizeof(char *));
     for (ac = 0; cp != NULL && ac < nargc; ac++) {
@@ -1369,7 +956,7 @@ resolve_editor(char *editor, int nfiles, char **files, char ***argv_out)
     nargv[ac] = NULL;
 
     *argv_out = nargv;
-    return editor_path;
+    debug_return_str(editor_path);
 }
 
 /*
@@ -1380,7 +967,10 @@ resolve_editor(char *editor, int nfiles, char **files, char ***argv_out)
 static char *
 find_editor(int nfiles, char **files, char ***argv_out)
 {
-    char *cp, *editor, *editor_path = NULL, **ev, *ev0[4];
+    const char *cp, *ep, *editor;
+    char *editor_path = NULL, **ev, *ev0[4];
+    size_t len;
+    debug_decl(find_editor, SUDO_DEBUG_PLUGIN)
 
     /*
      * If any of SUDO_EDITOR, VISUAL or EDITOR are set, choose the first one.
@@ -1389,29 +979,29 @@ find_editor(int nfiles, char **files, char ***argv_out)
     ev0[1] = "VISUAL";
     ev0[2] = "EDITOR";
     ev0[3] = NULL;
-    for (ev = ev0; *ev != NULL; ev++) {
+    for (ev = ev0; editor_path == NULL && *ev != NULL; ev++) {
        if ((editor = getenv(*ev)) != NULL && *editor != '\0') {
-           editor_path = resolve_editor(editor, nfiles, files, argv_out);
-           if (editor_path != NULL)
-               break;
+           editor_path = resolve_editor(editor, strlen(editor), nfiles,
+               files, argv_out);
        }
     }
     if (editor_path == NULL) {
-       /* def_editor could be a path, split it up */
-       editor = estrdup(def_editor);
-       cp = strtok(editor, ":");
-       while (cp != NULL && editor_path == NULL) {
-           editor_path = resolve_editor(cp, nfiles, files, argv_out);
-           cp = strtok(NULL, ":");
-       }
-       if (editor_path)
-           efree(editor);
+       /* def_editor could be a path, split it up, avoiding strtok() */
+       cp = editor = def_editor;
+       do {
+           if ((ep = strchr(cp, ':')) != NULL)
+               len = ep - cp;
+           else
+               len = strlen(cp);
+           editor_path = resolve_editor(cp, len, nfiles, files, argv_out);
+           cp = ep + 1;
+       } while (ep != NULL && editor_path == NULL);
     }
     if (!editor_path) {
-       audit_failure(NewArgv, "%s: command not found", editor);
-       warningx("%s: command not found", editor);
+       audit_failure(NewArgv, N_("%s: command not found"), editor);
+       warningx(_("%s: command not found"), editor);
     }
-    return editor_path;
+    debug_return_str(editor_path);
 }
 
 #ifdef USE_ADMIN_FLAG
@@ -1421,16 +1011,17 @@ create_admin_success_flag(void)
     struct stat statbuf;
     char flagfile[PATH_MAX];
     int fd, n;
+    debug_decl(create_admin_success_flag, SUDO_DEBUG_PLUGIN)
 
     /* Check whether the user is in the admin group. */
     if (!user_in_group(sudo_user.pw, "admin"))
-       return;
+       debug_return;
 
     /* Build path to flag file. */
     n = snprintf(flagfile, sizeof(flagfile), "%s/.sudo_as_admin_successful",
        user_dir);
     if (n <= 0 || n >= sizeof(flagfile))
-       return;
+       debug_return;
 
     /* Create admin flag file if it doesn't already exist. */
     set_perms(PERM_USER);
@@ -1439,6 +1030,7 @@ create_admin_success_flag(void)
        close(fd);
     }
     restore_perms();
+    debug_return;
 }
 #else /* !USE_ADMIN_FLAG */
 static void
@@ -1448,15 +1040,15 @@ create_admin_success_flag(void)
 }
 #endif /* USE_ADMIN_FLAG */
 
-struct policy_plugin sudoers_policy = {
-    SUDO_POLICY_PLUGIN,
-    SUDO_API_VERSION,
-    sudoers_policy_open,
-    sudoers_policy_close,
-    sudoers_policy_version,
-    sudoers_policy_check,
-    sudoers_policy_list,
-    sudoers_policy_validate,
-    sudoers_policy_invalidate,
-    sudoers_policy_init_session
-};
+static bool
+tty_present(void)
+{
+#if defined(HAVE_STRUCT_KINFO_PROC2_P_TDEV) || defined(HAVE_STRUCT_KINFO_PROC_P_TDEV) || defined(HAVE_STRUCT_KINFO_PROC_KI_TDEV) || defined(HAVE_STRUCT_KINFO_PROC_KP_EPROC_E_TDEV) || defined(HAVE_STRUCT_PSINFO_PR_TTYDEV) || defined(HAVE_PSTAT_GETPROC) || defined(__linux__)
+    return user_ttypath != NULL;
+#else
+    int fd = open(_PATH_TTY, O_RDWR|O_NOCTTY);
+    if (fd != -1)
+       close(fd);
+    return fd != -1;
+#endif
+}