]> granicus.if.org Git - sudo/commitdiff
s/rval/ret/g -- old habits die hard
authorTodd C. Miller <Todd.Miller@courtesan.com>
Tue, 1 Nov 2016 18:34:24 +0000 (12:34 -0600)
committerTodd C. Miller <Todd.Miller@courtesan.com>
Tue, 1 Nov 2016 18:34:24 +0000 (12:34 -0600)
plugins/sudoers/defaults.c
plugins/sudoers/mkdir_parents.c

index a4c1f705e66e931498ed3107b965b0d35e530d55..1d6564873f2f7895bd28a3cb5ba07063a2adf543 100644 (file)
@@ -365,7 +365,7 @@ bool
 run_early_defaults(void)
 {
     struct early_default *early;
-    bool rc = true;
+    bool ret = true;
     debug_decl(run_early_defaults, SUDOERS_DEBUG_DEFAULTS)
 
     for (early = early_defaults; early->var != NULL; early++) {
@@ -373,11 +373,11 @@ run_early_defaults(void)
            continue;
        if (early->def->callback != NULL) {
            if (!early->def->callback(&early->def->sd_un))
-               rc = false;
+               ret = false;
        }
        early->def = NULL;
     }
-    debug_return_bool(rc);
+    debug_return_bool(ret);
 }
 
 static void
@@ -656,7 +656,7 @@ bool
 update_defaults(int what, bool quiet)
 {
     struct defaults *def;
-    bool rc = true;
+    bool ret = true;
     debug_decl(update_defaults, SUDOERS_DEBUG_DEFAULTS)
 
     sudo_debug_printf(SUDO_DEBUG_INFO|SUDO_DEBUG_LINENO,
@@ -674,10 +674,10 @@ update_defaults(int what, bool quiet)
            !default_binding_matches(def, what))
            continue;
        if (!set_early_default(def->var, def->val, def->op, quiet, early))
-           rc = false;
+           ret = false;
     }
     if (!run_early_defaults())
-       rc = false;
+       ret = false;
 
     /*
      * Then set the rest of the defaults.
@@ -691,9 +691,9 @@ update_defaults(int what, bool quiet)
            !default_binding_matches(def, what))
            continue;
        if (!set_default(def->var, def->val, def->op, quiet))
-           rc = false;
+           ret = false;
     }
-    debug_return_bool(rc);
+    debug_return_bool(ret);
 }
 
 /*
@@ -705,7 +705,7 @@ check_defaults(int what, bool quiet)
 {
     struct sudo_defs_types *cur, tmp;
     struct defaults *def;
-    bool rc = true;
+    bool ret = true;
     debug_decl(check_defaults, SUDOERS_DEBUG_DEFAULTS)
 
     TAILQ_FOREACH(def, &defaults, entries) {
@@ -718,17 +718,17 @@ check_defaults(int what, bool quiet)
        if (cur->name == NULL) {
            if (!quiet)
                sudo_warnx(U_("unknown defaults entry `%s'"), def->var);
-           rc = false;
+           ret = false;
        } else {
            /* Don't actually set the defaults value, just checking. */
            tmp = *cur;
            memset(&tmp.sd_un, 0, sizeof(tmp.sd_un));
            if (!set_default_entry(&tmp, def->val, def->op, quiet, false))
-               rc = false;
+               ret = false;
            free_default(&tmp);
        }
     }
-    debug_return_bool(rc);
+    debug_return_bool(ret);
 }
 
 static bool
index 53f1feb72bbf73f77bc94ba3ec0c7f1c66d9cc34..c915eae714643e575c1bf9322bbfe9c518cfa43f 100644 (file)
@@ -42,7 +42,7 @@ sudo_mkdir_parents(char *path, uid_t uid, gid_t *gidp, mode_t mode, bool quiet)
     struct stat sb;
     gid_t parent_gid = 0;
     char *slash = path;
-    bool rval = true;
+    bool ret = true;
     debug_decl(sudo_mkdir_parents, SUDOERS_DEBUG_UTIL)
 
     /* If no gid specified, inherit from parent dir. */
@@ -63,21 +63,21 @@ sudo_mkdir_parents(char *path, uid_t uid, gid_t *gidp, mode_t mode, bool quiet)
            if (errno != EEXIST) {
                if (!quiet)
                    sudo_warn(U_("unable to mkdir %s"), path);
-               rval = false;
+               ret = false;
                break;
            }
            /* Already exists, make sure it is a directory. */
            if (stat(path, &sb) != 0) {
                if (!quiet)
                    sudo_warn(U_("unable to stat %s"), path);
-               rval = false;
+               ret = false;
                break;
            }
            if (!S_ISDIR(sb.st_mode)) {
                if (!quiet)
                    sudo_warnx(U_("%s exists but is not a directory (0%o)"),
                        path, (unsigned int) sb.st_mode);
-               rval = false;
+               ret = false;
                break;
            }
            /* Inherit gid of parent dir for ownership. */
@@ -88,7 +88,7 @@ sudo_mkdir_parents(char *path, uid_t uid, gid_t *gidp, mode_t mode, bool quiet)
     }
 
     /* Return parent gid if none was specified by caller. */
-    if (rval && *gidp == (gid_t)-1)
+    if (ret && *gidp == (gid_t)-1)
        *gidp = parent_gid;
-    debug_return_bool(rval);
+    debug_return_bool(ret);
 }