Imported Upstream version 1.7.6p1
[debian/sudo] / defaults.c
index 4621bbef9e144b388f24faa7e54fc25d5d31ecf0..c1f0afa76d66bc753467d8913ebddbb52154f321 100644 (file)
@@ -1,38 +1,25 @@
 /*
- * Copyright (c) 1999-2001 Todd C. Miller <Todd.Miller@courtesan.com>
- * All rights reserved.
+ * Copyright (c) 1999-2005, 2007-2008, 2010
+ *     Todd C. Miller <Todd.Miller@courtesan.com>
  *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
+ * Permission to use, copy, modify, and distribute this software for any
+ * purpose with or without fee is hereby granted, provided that the above
+ * copyright notice and this permission notice appear in all copies.
  *
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
+ * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
+ * WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
+ * MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
+ * ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
+ * WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
+ * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
+ * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  *
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- *
- * 3. The name of the author may not be used to endorse or promote products
- *    derived from this software without specific prior written permission.
- *
- * 4. Products derived from this software may not be called "Sudo" nor
- *    may "Sudo" appear in their names without specific prior written
- *    permission from the author.
- *
- * THIS SOFTWARE IS PROVIDED ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES,
- * INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY
- * AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED.  IN NO EVENT SHALL
- * THE AUTHOR BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
- * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
- * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS;
- * OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY,
- * WHETHER IN CONTRACT, STRICT 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.
+ * 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.
  */
 
-#include "config.h"
+#include <config.h>
 
 #include <sys/types.h>
 #include <sys/param.h>
 #endif /* STDC_HEADERS */
 #ifdef HAVE_STRING_H
 # include <string.h>
-#else
-# ifdef HAVE_STRINGS_H
-#  include <strings.h>
-# endif
 #endif /* HAVE_STRING_H */
+#ifdef HAVE_STRINGS_H
+# include <strings.h>
+#endif /* HAVE_STRINGS_H */
 # ifdef HAVE_UNISTD_H
 #include <unistd.h>
 #endif /* HAVE_UNISTD_H */
+#include <pwd.h>
 #include <ctype.h>
 
 #include "sudo.h"
-
-#ifndef lint
-static const char rcsid[] = "$Sudo: defaults.c,v 1.38 2001/12/30 18:40:09 millert Exp $";
-#endif /* lint */
+#include "parse.h"
+#include <gram.h>
 
 /*
  * For converting between syslog numbers and strings.
@@ -103,20 +88,21 @@ static struct strmap priorities[] = {
        { NULL,         -1 }
 };
 
-extern int sudolineno;
-
 /*
  * Local prototypes.
  */
 static int store_int __P((char *, struct sudo_defs_types *, int));
-static int store_uint __P((char *, struct sudo_defs_types *, int));
+static int store_list __P((char *, struct sudo_defs_types *, int));
+static int store_mode __P((char *, struct sudo_defs_types *, int));
 static int store_str __P((char *, struct sudo_defs_types *, int));
 static int store_syslogfac __P((char *, struct sudo_defs_types *, int));
 static int store_syslogpri __P((char *, struct sudo_defs_types *, int));
-static int store_mode __P((char *, struct sudo_defs_types *, int));
-static int store_pwflag __P((char *, struct sudo_defs_types *, int));
-static int store_list __P((char *, struct sudo_defs_types *, int));
+static int store_tuple __P((char *, struct sudo_defs_types *, int));
+static int store_uint __P((char *, struct sudo_defs_types *, int));
+static int store_float __P((char *, struct sudo_defs_types *, int));
 static void list_op __P((char *, size_t, struct sudo_defs_types *, enum list_ops));
+static const char *logfac2str __P((int));
+static const char *logpri2str __P((int));
 
 /*
  * Table describing compile-time and run-time options.
@@ -131,6 +117,7 @@ dump_defaults()
 {
     struct sudo_defs_types *cur;
     struct list_member *item;
+    struct def_values *def;
 
     for (cur = sudo_defs_table; cur->name; cur++) {
        if (cur->desc) {
@@ -140,19 +127,32 @@ dump_defaults()
                        puts(cur->desc);
                    break;
                case T_STR:
-               case T_LOGFAC:
-               case T_LOGPRI:
-               case T_PWFLAG:
                    if (cur->sd_un.str) {
                        (void) printf(cur->desc, cur->sd_un.str);
                        putchar('\n');
                    }
                    break;
+               case T_LOGFAC:
+                   if (cur->sd_un.ival) {
+                       (void) printf(cur->desc, logfac2str(cur->sd_un.ival));
+                       putchar('\n');
+                   }
+                   break;
+               case T_LOGPRI:
+                   if (cur->sd_un.ival) {
+                       (void) printf(cur->desc, logpri2str(cur->sd_un.ival));
+                       putchar('\n');
+                   }
+                   break;
                case T_UINT:
                case T_INT:
                    (void) printf(cur->desc, cur->sd_un.ival);
                    putchar('\n');
                    break;
+               case T_FLOAT:
+                   (void) printf(cur->desc, cur->sd_un.fval);
+                   putchar('\n');
+                   break;
                case T_MODE:
                    (void) printf(cur->desc, cur->sd_un.mode);
                    putchar('\n');
@@ -164,6 +164,15 @@ dump_defaults()
                            printf("\t%s\n", item->value);
                    }
                    break;
+               case T_TUPLE:
+                   for (def = cur->values; def->sval; def++) {
+                       if (cur->sd_un.ival == def->ival) {
+                           (void) printf(cur->desc, def->sval);
+                           break;
+                       }
+                   }
+                   putchar('\n');
+                   break;
            }
        }
     }
@@ -219,158 +228,131 @@ set_default(var, val, op)
            break;
     }
     if (!cur->name) {
-       (void) fprintf(stderr,
-           "%s: unknown defaults entry `%s' referenced near line %d\n", Argv[0],
-           var, sudolineno);
-       return(FALSE);
+       warningx("unknown defaults entry `%s'", var);
+       return FALSE;
     }
 
     switch (cur->type & T_MASK) {
        case T_LOGFAC:
            if (!store_syslogfac(val, cur, op)) {
                if (val)
-                   (void) fprintf(stderr,
-                       "%s: value '%s' is invalid for option '%s'\n", Argv[0],
-                       val, var);
+                   warningx("value `%s' is invalid for option `%s'", val, var);
                else
-                   (void) fprintf(stderr,
-                       "%s: no value specified for `%s' on line %d\n", Argv[0],
-                       var, sudolineno);
-               return(FALSE);
+                   warningx("no value specified for `%s'", var);
+               return FALSE;
            }
            break;
        case T_LOGPRI:
            if (!store_syslogpri(val, cur, op)) {
                if (val)
-                   (void) fprintf(stderr,
-                       "%s: value '%s' is invalid for option '%s'\n", Argv[0],
-                       val, var);
-               else
-                   (void) fprintf(stderr,
-                       "%s: no value specified for `%s' on line %d\n", Argv[0],
-                       var, sudolineno);
-               return(FALSE);
-           }
-           break;
-       case T_PWFLAG:
-           if (!store_pwflag(val, cur, op)) {
-               if (val)
-                   (void) fprintf(stderr,
-                       "%s: value '%s' is invalid for option '%s'\n", Argv[0],
-                       val, var);
+                   warningx("value `%s' is invalid for option `%s'", val, var);
                else
-                   (void) fprintf(stderr,
-                       "%s: no value specified for `%s' on line %d\n", Argv[0],
-                       var, sudolineno);
-               return(FALSE);
+                   warningx("no value specified for `%s'", var);
+               return FALSE;
            }
            break;
        case T_STR:
            if (!val) {
                /* Check for bogus boolean usage or lack of a value. */
-               if (!(cur->type & T_BOOL) || op != FALSE) {
-                   (void) fprintf(stderr,
-                       "%s: no value specified for `%s' on line %d\n", Argv[0],
-                       var, sudolineno);
-                   return(FALSE);
+               if (!ISSET(cur->type, T_BOOL) || op != FALSE) {
+                   warningx("no value specified for `%s'", var);
+                   return FALSE;
                }
            }
-           if ((cur->type & T_PATH) && val && *val != '/') {
-               (void) fprintf(stderr,
-                   "%s: values for `%s' must start with a '/'\n", Argv[0],
-                   var);
-               return(FALSE);
+           if (ISSET(cur->type, T_PATH) && val && *val != '/') {
+               warningx("values for `%s' must start with a '/'", var);
+               return FALSE;
            }
            if (!store_str(val, cur, op)) {
-               (void) fprintf(stderr,
-                   "%s: value '%s' is invalid for option '%s'\n", Argv[0],
-                   val, var);
-               return(FALSE);
+               warningx("value `%s' is invalid for option `%s'", val, var);
+               return FALSE;
            }
            break;
        case T_INT:
            if (!val) {
                /* Check for bogus boolean usage or lack of a value. */
-               if (!(cur->type & T_BOOL) || op != FALSE) {
-                   (void) fprintf(stderr,
-                       "%s: no value specified for `%s' on line %d\n", Argv[0],
-                       var, sudolineno);
-                   return(FALSE);
+               if (!ISSET(cur->type, T_BOOL) || op != FALSE) {
+                   warningx("no value specified for `%s'", var);
+                   return FALSE;
                }
            }
            if (!store_int(val, cur, op)) {
-               (void) fprintf(stderr,
-                   "%s: value '%s' is invalid for option '%s'\n", Argv[0],
-                   val, var);
-               return(FALSE);
+               warningx("value `%s' is invalid for option `%s'", val, var);
+               return FALSE;
            }
            break;
        case T_UINT:
            if (!val) {
                /* Check for bogus boolean usage or lack of a value. */
-               if (!(cur->type & T_BOOL) || op != FALSE) {
-                   (void) fprintf(stderr,
-                       "%s: no value specified for `%s' on line %d\n", Argv[0],
-                       var, sudolineno);
-                   return(FALSE);
+               if (!ISSET(cur->type, T_BOOL) || op != FALSE) {
+                   warningx("no value specified for `%s'", var);
+                   return FALSE;
                }
            }
            if (!store_uint(val, cur, op)) {
-               (void) fprintf(stderr,
-                   "%s: value '%s' is invalid for option '%s'\n", Argv[0],
-                   val, var);
-               return(FALSE);
+               warningx("value `%s' is invalid for option `%s'", val, var);
+               return FALSE;
+           }
+           break;
+       case T_FLOAT:
+           if (!val) {
+               /* Check for bogus boolean usage or lack of a value. */
+               if (!ISSET(cur->type, T_BOOL) || op != FALSE) {
+                   warningx("no value specified for `%s'", var);
+                   return FALSE;
+               }
+           }
+           if (!store_float(val, cur, op)) {
+               warningx("value `%s' is invalid for option `%s'", val, var);
+               return FALSE;
            }
            break;
        case T_MODE:
            if (!val) {
                /* Check for bogus boolean usage or lack of a value. */
-               if (!(cur->type & T_BOOL) || op != FALSE) {
-                   (void) fprintf(stderr,
-                       "%s: no value specified for `%s' on line %d\n", Argv[0],
-                       var, sudolineno);
-                   return(FALSE);
+               if (!ISSET(cur->type, T_BOOL) || op != FALSE) {
+                   warningx("no value specified for `%s'", var);
+                   return FALSE;
                }
            }
            if (!store_mode(val, cur, op)) {
-               (void) fprintf(stderr,
-                   "%s: value '%s' is invalid for option '%s'\n", Argv[0],
-                   val, var);
-               return(FALSE);
+               warningx("value `%s' is invalid for option `%s'", val, var);
+               return FALSE;
            }
            break;
        case T_FLAG:
            if (val) {
-               (void) fprintf(stderr,
-                   "%s: option `%s' does not take a value on line %d\n",
-                   Argv[0], var, sudolineno);
-               return(FALSE);
+               warningx("option `%s' does not take a value", var);
+               return FALSE;
            }
            cur->sd_un.flag = op;
-
-           /* Special action for I_FQDN.  Move to own switch if we get more */
-           if (num == I_FQDN && op)
-               set_fqdn();
            break;
        case T_LIST:
            if (!val) {
                /* Check for bogus boolean usage or lack of a value. */
-               if (!(cur->type & T_BOOL) || op != FALSE) {
-                   (void) fprintf(stderr,
-                       "%s: no value specified for `%s' on line %d\n", Argv[0],
-                       var, sudolineno);
-                   return(FALSE);
+               if (!ISSET(cur->type, T_BOOL) || op != FALSE) {
+                   warningx("no value specified for `%s'", var);
+                   return FALSE;
                }
            }
            if (!store_list(val, cur, op)) {
-               (void) fprintf(stderr,
-                   "%s: value '%s' is invalid for option '%s'\n", Argv[0],
-                   val, var);
-               return(FALSE);
+               warningx("value `%s' is invalid for option `%s'", val, var);
+               return FALSE;
+           }
+           break;
+       case T_TUPLE:
+           if (!val && !ISSET(cur->type, T_BOOL)) {
+               warningx("no value specified for `%s'", var);
+               return FALSE;
+           }
+           if (!store_tuple(val, cur, op)) {
+               warningx("value `%s' is invalid for option `%s'", val, var);
+               return FALSE;
            }
+           break;
     }
 
-    return(TRUE);
+    return TRUE;
 }
 
 /*
@@ -383,78 +365,85 @@ init_defaults()
     static int firsttime = 1;
     struct sudo_defs_types *def;
 
-    /* Free any strings that were set. */
+    /* Clear any old settings. */
     if (!firsttime) {
-       for (def = sudo_defs_table; def->name; def++)
+       for (def = sudo_defs_table; def->name; def++) {
            switch (def->type & T_MASK) {
                case T_STR:
-               case T_LOGFAC:
-               case T_LOGPRI:
-               case T_PWFLAG:
-                   if (def->sd_un.str) {
-                       free(def->sd_un.str);
-                       def->sd_un.str = NULL;
-                   }
+                   efree(def->sd_un.str);
+                   def->sd_un.str = NULL;
                    break;
                case T_LIST:
                    list_op(NULL, 0, def, freeall);
                    break;
            }
+           zero_bytes(&def->sd_un, sizeof(def->sd_un));
+       }
     }
 
     /* First initialize the flags. */
 #ifdef LONG_OTP_PROMPT
-    def_flag(I_LONG_OTP_PROMPT) = TRUE;
+    def_long_otp_prompt = TRUE;
 #endif
 #ifdef IGNORE_DOT_PATH
-    def_flag(I_IGNORE_DOT) = TRUE;
+    def_ignore_dot = TRUE;
 #endif
 #ifdef ALWAYS_SEND_MAIL
-    def_flag(I_MAIL_ALWAYS) = TRUE;
+    def_mail_always = TRUE;
 #endif
 #ifdef SEND_MAIL_WHEN_NO_USER
-    def_flag(I_MAIL_NO_USER) = TRUE;
+    def_mail_no_user = TRUE;
 #endif
 #ifdef SEND_MAIL_WHEN_NO_HOST
-    def_flag(I_MAIL_NO_HOST) = TRUE;
+    def_mail_no_host = TRUE;
 #endif
 #ifdef SEND_MAIL_WHEN_NOT_OK
-    def_flag(I_MAIL_NO_PERMS) = TRUE;
+    def_mail_no_perms = TRUE;
 #endif
-#ifdef USE_TTY_TICKETS
-    def_flag(I_TTY_TICKETS) = TRUE;
+#ifndef NO_TTY_TICKETS
+    def_tty_tickets = TRUE;
 #endif
 #ifndef NO_LECTURE
-    def_flag(I_LECTURE) = TRUE;
+    def_lecture = once;
 #endif
 #ifndef NO_AUTHENTICATION
-    def_flag(I_AUTHENTICATE) = TRUE;
+    def_authenticate = TRUE;
 #endif
 #ifndef NO_ROOT_SUDO
-    def_flag(I_ROOT_SUDO) = TRUE;
+    def_root_sudo = TRUE;
 #endif
 #ifdef HOST_IN_LOG
-    def_flag(I_LOG_HOST) = TRUE;
+    def_log_host = TRUE;
 #endif
 #ifdef SHELL_IF_NO_ARGS
-    def_flag(I_SHELL_NOARGS) = TRUE;
+    def_shell_noargs = TRUE;
 #endif
 #ifdef SHELL_SETS_HOME
-    def_flag(I_SET_HOME) = TRUE;
+    def_set_home = TRUE;
 #endif
 #ifndef DONT_LEAK_PATH_INFO
-    def_flag(I_PATH_INFO) = TRUE;
+    def_path_info = TRUE;
 #endif
 #ifdef FQDN
-    def_flag(I_FQDN) = TRUE;
+    def_fqdn = TRUE;
 #endif
 #ifdef USE_INSULTS
-    def_flag(I_INSULTS) = TRUE;
+    def_insults = TRUE;
 #endif
 #ifdef ENV_EDITOR
-    def_flag(I_ENV_EDITOR) = TRUE;
+    def_env_editor = TRUE;
+#endif
+#ifdef UMASK_OVERRIDE
+    def_umask_override = TRUE;
 #endif
-    def_flag(I_SET_LOGNAME) = TRUE;
+#ifdef _PATH_SUDO_ASKPASS
+    def_askpass = estrdup(_PATH_SUDO_ASKPASS);
+#endif
+    def_iolog_dir = estrdup(_PATH_SUDO_IO_LOGDIR);
+    def_sudoers_locale = estrdup("C");
+    def_env_reset = ENV_RESET;
+    def_set_logname = TRUE;
+    def_closefrom = STDERR_FILENO + 1;
 
     /* Syslog options need special care since they both strings and ints */
 #if (LOGGING & SLOG_SYSLOG)
@@ -466,53 +455,101 @@ init_defaults()
 #endif
 
     /* Password flags also have a string and integer component. */
-    (void) store_pwflag("any", &sudo_defs_table[I_LISTPW], TRUE);
-    (void) store_pwflag("all", &sudo_defs_table[I_VERIFYPW], TRUE);
+    (void) store_tuple("any", &sudo_defs_table[I_LISTPW], TRUE);
+    (void) store_tuple("all", &sudo_defs_table[I_VERIFYPW], TRUE);
 
     /* Then initialize the int-like things. */
 #ifdef SUDO_UMASK
-    def_mode(I_UMASK) = SUDO_UMASK;
+    def_umask = SUDO_UMASK;
 #else
-    def_mode(I_UMASK) = 0777;
+    def_umask = 0777;
+#endif
+    def_loglinelen = MAXLOGFILELEN;
+    def_timestamp_timeout = TIMEOUT;
+    def_passwd_timeout = PASSWORD_TIMEOUT;
+    def_passwd_tries = TRIES_FOR_PASSWORD;
+#ifdef HAVE_ZLIB_H
+    def_compress_io = TRUE;
 #endif
-    def_ival(I_LOGLINELEN) = MAXLOGFILELEN;
-    def_ival(I_TIMESTAMP_TIMEOUT) = TIMEOUT;
-    def_ival(I_PASSWD_TIMEOUT) = PASSWORD_TIMEOUT;
-    def_ival(I_PASSWD_TRIES) = TRIES_FOR_PASSWORD;
 
     /* Now do the strings */
-    def_str(I_MAILTO) = estrdup(MAILTO);
-    def_str(I_MAILSUB) = estrdup(MAILSUBJECT);
-    def_str(I_BADPASS_MESSAGE) = estrdup(INCORRECT_PASSWORD);
-    def_str(I_TIMESTAMPDIR) = estrdup(_PATH_SUDO_TIMEDIR);
-    def_str(I_PASSPROMPT) = estrdup(PASSPROMPT);
-    def_str(I_RUNAS_DEFAULT) = estrdup(RUNAS_DEFAULT);
+    def_mailto = estrdup(MAILTO);
+    def_mailsub = estrdup(MAILSUBJECT);
+    def_badpass_message = estrdup(INCORRECT_PASSWORD);
+    def_timestampdir = estrdup(_PATH_SUDO_TIMEDIR);
+    def_passprompt = estrdup(PASSPROMPT);
+    def_runas_default = estrdup(RUNAS_DEFAULT);
 #ifdef _PATH_SUDO_SENDMAIL
-    def_str(I_MAILERPATH) = estrdup(_PATH_SUDO_SENDMAIL);
-    def_str(I_MAILERFLAGS) = estrdup("-t");
+    def_mailerpath = estrdup(_PATH_SUDO_SENDMAIL);
+    def_mailerflags = estrdup("-t");
 #endif
 #if (LOGGING & SLOG_FILE)
-    def_str(I_LOGFILE) = estrdup(_PATH_SUDO_LOGFILE);
+    def_logfile = estrdup(_PATH_SUDO_LOGFILE);
 #endif
 #ifdef EXEMPTGROUP
-    def_str(I_EXEMPT_GROUP) = estrdup(EXEMPTGROUP);
+    def_exempt_group = estrdup(EXEMPTGROUP);
+#endif
+#ifdef SECURE_PATH
+    def_secure_path = estrdup(SECURE_PATH);
+#endif
+    def_editor = estrdup(EDITOR);
+#ifdef _PATH_SUDO_NOEXEC
+    def_noexec_file = estrdup(_PATH_SUDO_NOEXEC);
 #endif
-    def_str(I_EDITOR) = estrdup(EDITOR);
 
     /* Finally do the lists (currently just environment tables). */
     init_envtables();
 
-    /*
-     * The following depend on the above values.
-     * We use a pointer to the string so that if its
-     * value changes we get the change.
-     */
-    if (user_runas == NULL)
-       user_runas = &def_str(I_RUNAS_DEFAULT);
-
     firsttime = 0;
 }
 
+/*
+ * Update the defaults based on what was set by sudoers.
+ * Pass in an OR'd list of which default types to update.
+ */
+int
+update_defaults(what)
+    int what;
+{
+    struct defaults *def;
+    int rc = TRUE;
+
+    tq_foreach_fwd(&defaults, def) {
+       switch (def->type) {
+           case DEFAULTS:
+               if (ISSET(what, SETDEF_GENERIC) &&
+                   !set_default(def->var, def->val, def->op))
+                   rc = FALSE;
+               break;
+           case DEFAULTS_USER:
+               if (ISSET(what, SETDEF_USER) &&
+                   userlist_matches(sudo_user.pw, &def->binding) == ALLOW &&
+                   !set_default(def->var, def->val, def->op))
+                   rc = FALSE;
+               break;
+           case DEFAULTS_RUNAS:
+               if (ISSET(what, SETDEF_RUNAS) &&
+                   runaslist_matches(&def->binding, NULL) == ALLOW &&
+                   !set_default(def->var, def->val, def->op))
+                   rc = FALSE;
+               break;
+           case DEFAULTS_HOST:
+               if (ISSET(what, SETDEF_HOST) &&
+                   hostlist_matches(&def->binding) == ALLOW &&
+                   !set_default(def->var, def->val, def->op))
+                   rc = FALSE;
+               break;
+           case DEFAULTS_CMND:
+               if (ISSET(what, SETDEF_CMND) &&
+                   cmndlist_matches(&def->binding) == ALLOW &&
+                   !set_default(def->var, def->val, def->op))
+                   rc = FALSE;
+               break;
+       }
+    }
+    return rc;
+}
+
 static int
 store_int(val, def, op)
     char *val;
@@ -527,11 +564,13 @@ store_int(val, def, op)
     } else {
        l = strtol(val, &endp, 10);
        if (*endp != '\0')
-           return(FALSE);
+           return FALSE;
        /* XXX - should check against INT_MAX */
-       def->sd_un.ival = (unsigned int)l;
+       def->sd_un.ival = (int)l;
     }
-    return(TRUE);
+    if (def->callback)
+       return def->callback(val);
+    return TRUE;
 }
 
 static int
@@ -548,11 +587,68 @@ store_uint(val, def, op)
     } else {
        l = strtol(val, &endp, 10);
        if (*endp != '\0' || l < 0)
-           return(FALSE);
+           return FALSE;
        /* XXX - should check against INT_MAX */
        def->sd_un.ival = (unsigned int)l;
     }
-    return(TRUE);
+    if (def->callback)
+       return def->callback(val);
+    return TRUE;
+}
+
+static int
+store_float(val, def, op)
+    char *val;
+    struct sudo_defs_types *def;
+    int op;
+{
+    char *endp;
+    double d;
+
+    if (op == FALSE) {
+       def->sd_un.fval = 0.0;
+    } else {
+       d = strtod(val, &endp);
+       if (*endp != '\0')
+           return FALSE;
+       /* XXX - should check against HUGE_VAL */
+       def->sd_un.fval = d;
+    }
+    if (def->callback)
+       return def->callback(val);
+    return TRUE;
+}
+
+static int
+store_tuple(val, def, op)
+    char *val;
+    struct sudo_defs_types *def;
+    int op;
+{
+    struct def_values *v;
+
+    /*
+     * Since enums are really just ints we store the value as an ival.
+     * In the future, there may be multiple enums for different tuple
+     * types we want to avoid and special knowledge of the tuple type.
+     * This does assume that the first entry in the tuple enum will
+     * be the equivalent to a boolean "false".
+     */
+    if (!val) {
+       def->sd_un.ival = (op == FALSE) ? 0 : 1;
+    } else {
+       for (v = def->values; v->sval != NULL; v++) {
+           if (strcmp(v->sval, val) == 0) {
+               def->sd_un.ival = v->ival;
+               break;
+           }
+       }
+       if (v->sval == NULL)
+           return FALSE;
+    }
+    if (def->callback)
+       return def->callback(val);
+    return TRUE;
 }
 
 static int
@@ -562,13 +658,14 @@ store_str(val, def, op)
     int op;
 {
 
-    if (def->sd_un.str)
-       free(def->sd_un.str);
+    efree(def->sd_un.str);
     if (op == FALSE)
        def->sd_un.str = NULL;
     else
        def->sd_un.str = estrdup(val);
-    return(TRUE);
+    if (def->callback)
+       return def->callback(val);
+    return TRUE;
 }
 
 static int
@@ -588,18 +685,18 @@ store_list(str, def, op)
        end = str;
        do {
            /* Remove leading blanks, if nothing but blanks we are done. */
-           for (start = end; isblank(*start); start++)
+           for (start = end; isblank((unsigned char)*start); start++)
                ;
            if (*start == '\0')
                break;
 
            /* Find end position and perform operation. */
-           for (end = start; *end && !isblank(*end); end++) 
+           for (end = start; *end && !isblank((unsigned char)*end); end++)
                ;
            list_op(start, end - start, def, op == '-' ? delete : add);
        } while (*end++ != '\0');
     }
-    return(TRUE);
+    return TRUE;
 }
 
 static int
@@ -611,31 +708,37 @@ store_syslogfac(val, def, op)
     struct strmap *fac;
 
     if (op == FALSE) {
-       if (def->sd_un.str) {
-           free(def->sd_un.str);
-           def->sd_un.str = NULL;
-       }
-       return(TRUE);
+       def->sd_un.ival = FALSE;
+       return TRUE;
     }
 #ifdef LOG_NFACILITIES
     if (!val)
-       return(FALSE);
+       return FALSE;
     for (fac = facilities; fac->name && strcmp(val, fac->name); fac++)
        ;
     if (fac->name == NULL)
-       return(FALSE);                          /* not found */
+       return FALSE;                           /* not found */
+
+    def->sd_un.ival = fac->num;
+#else
+    def->sd_un.ival = -1;
+#endif /* LOG_NFACILITIES */
+    return TRUE;
+}
 
-    /* Store both name and number. */
-    if (def->sd_un.str)
-       free(def->sd_un.str);
-    def->sd_un.str = estrdup(fac->name);
-    sudo_defs_table[I_LOGFAC].sd_un.ival = fac->num;
+static const char *
+logfac2str(n)
+    int n;
+{
+#ifdef LOG_NFACILITIES
+    struct strmap *fac;
+
+    for (fac = facilities; fac->name && fac->num != n; fac++)
+       ;
+    return fac->name;
 #else
-    if (def->sd_un.str)
-       free(def->sd_un.str);
-    def->sd_un.str = estrdup("default");
+    return "default";
 #endif /* LOG_NFACILITIES */
-    return(TRUE);
 }
 
 static int
@@ -645,28 +748,28 @@ store_syslogpri(val, def, op)
     int op;
 {
     struct strmap *pri;
-    struct sudo_defs_types *idef;
 
     if (op == FALSE || !val)
-       return(FALSE);
-    if (def == &sudo_defs_table[I_SYSLOG_GOODPRI])
-       idef = &sudo_defs_table[I_GOODPRI];
-    else if (def == &sudo_defs_table[I_SYSLOG_BADPRI])
-       idef = &sudo_defs_table[I_BADPRI];
-    else
-       return(FALSE);
+       return FALSE;
 
     for (pri = priorities; pri->name && strcmp(val, pri->name); pri++)
        ;
     if (pri->name == NULL)
-       return(FALSE);                          /* not found */
-
-    /* Store both name and number. */
-    if (def->sd_un.str)
-       free(def->sd_un.str);
-    def->sd_un.str = estrdup(pri->name);
-    idef->sd_un.ival = pri->num;
-    return(TRUE);
+       return FALSE;                           /* not found */
+
+    def->sd_un.ival = pri->num;
+    return TRUE;
+}
+
+static const char *
+logpri2str(n)
+    int n;
+{
+    struct strmap *pri;
+
+    for (pri = priorities; pri->name && pri->num != n; pri++)
+       ;
+    return pri->name;
 }
 
 static int
@@ -683,57 +786,12 @@ store_mode(val, def, op)
     } else {
        l = strtol(val, &endp, 8);
        if (*endp != '\0' || l < 0 || l > 0777)
-           return(FALSE);
+           return FALSE;
        def->sd_un.mode = (mode_t)l;
     }
-    return(TRUE);
-}
-
-static int
-store_pwflag(val, def, op)
-    char *val;
-    struct sudo_defs_types *def;
-    int op;
-{
-    int isub, flags;
-
-    if (strcmp(def->name, "verifypw") == 0)
-       isub = I_VERIFYPW_I;
-    else
-       isub = I_LISTPW_I;
-
-    /* Handle !foo. */
-    if (op == FALSE) {
-       if (def->sd_un.str) {
-           free(def->sd_un.str);
-           def->sd_un.str = NULL;
-       }
-       def->sd_un.str = estrdup("never");
-       sudo_defs_table[isub].sd_un.ival = PWCHECK_NEVER;
-       return(TRUE);
-    }
-    if (!val)
-       return(FALSE);
-
-    /* Convert strings to integer values. */
-    if (strcmp(val, "all") == 0)
-       flags = PWCHECK_ALL;
-    else if (strcmp(val, "any") == 0)
-       flags = PWCHECK_ANY;
-    else if (strcmp(val, "never") == 0)
-       flags = PWCHECK_NEVER;
-    else if (strcmp(val, "always") == 0)
-       flags = PWCHECK_ALWAYS;
-    else
-       return(FALSE);
-
-    /* Store both name and number. */
-    if (def->sd_un.str)
-       free(def->sd_un.str);
-    def->sd_un.str = estrdup(val);
-    sudo_defs_table[isub].sd_un.ival = flags;
-
-    return(TRUE);
+    if (def->callback)
+       return def->callback(val);
+    return TRUE;
 }
 
 static void
@@ -749,8 +807,8 @@ list_op(val, len, def, op)
        for (cur = def->sd_un.list; cur; ) {
            tmp = cur;
            cur = tmp->next;
-           free(tmp->value);
-           free(tmp);
+           efree(tmp->value);
+           efree(tmp);
        }
        def->sd_un.list = NULL;
        return;
@@ -767,8 +825,8 @@ list_op(val, len, def, op)
                prev->next = cur->next;
            else
                def->sd_un.list = cur->next;
-           free(cur->value);
-           free(cur);
+           efree(cur->value);
+           efree(cur);
            break;
        }
     }