Flags always have a NULL value. Regression introduced by refactor

of set_default_entry().

diff --git a/plugins/sudoers/defaults.c b/plugins/sudoers/defaults.c
index ace368d393fdb37c3370acb09c36b31c1591d0b5..03e33bc74e4fa6cf7e04b135ebf48ae3ed3bf06f 100644 (file)
--- a/plugins/sudoers/defaults.c
+++ b/plugins/sudoers/defaults.c
@@ -196,7 +196,7 @@ set_default_entry(struct sudo_defs_types *def, const char *val, int op,
     int rc;
     debug_decl(set_default_entry, SUDOERS_DEBUG_DEFAULTS)
 
-    if (val == NULL) {
+    if (val == NULL && !ISSET(def->type, T_FLAG)) {
        /* Check for bogus boolean usage or missing value if non-boolean. */
        if (!ISSET(def->type, T_BOOL) || op != false) {
            if (!quiet)