diff options
| author | Pavel Březina <pbrezina@redhat.com> | 2012-02-06 10:43:56 +0100 |
|---|---|---|
| committer | Stephen Gallagher <sgallagh@redhat.com> | 2012-02-06 08:36:51 -0500 |
| commit | ed44814e0e7ff9f0ef7ffc98fab7d9542a7822df (patch) | |
| tree | 693c2ffd06f71a9b8fac0a62712154d5c784d2de | |
| parent | 699cb9781f64796af68ea620fd4952d47ac0d64a (diff) | |
| download | sssd-ed44814e0e7ff9f0ef7ffc98fab7d9542a7822df.tar.gz sssd-ed44814e0e7ff9f0ef7ffc98fab7d9542a7822df.tar.xz sssd-ed44814e0e7ff9f0ef7ffc98fab7d9542a7822df.zip | |
SUDO Integration - sysdb_sudo_check_time() fix
| -rw-r--r-- | src/db/sysdb_sudo.c | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/src/db/sysdb_sudo.c b/src/db/sysdb_sudo.c index 5f87a80cd..fbe7cc8c5 100644 --- a/src/db/sysdb_sudo.c +++ b/src/db/sysdb_sudo.c @@ -71,13 +71,14 @@ static errno_t sysdb_sudo_check_time(struct sysdb_attrs *rule, /* check for sudoNotBefore */ ret = sysdb_attrs_get_string_array(rule, SYSDB_SUDO_CACHE_AT_NOTBEFORE, tmp_ctx, &values); - if (ret != EOK) { - goto done; - } else if (ret == ENOENT) { + if (ret == ENOENT) { DEBUG(SSSDBG_TRACE_LIBS, ("notBefore attribute is missing, the rule is valid\n")); *result = true; ret = EOK; + goto done; + } else if (ret != EOK) { + goto done; } for (i=0; values[i] ; i++) { @@ -100,13 +101,14 @@ static errno_t sysdb_sudo_check_time(struct sysdb_attrs *rule, /* check for sudoNotAfter */ ret = sysdb_attrs_get_string_array(rule, SYSDB_SUDO_CACHE_AT_NOTAFTER, tmp_ctx, &values); - if (ret != EOK) { - goto done; - } else if (ret == ENOENT) { + if (ret == ENOENT) { DEBUG(SSSDBG_TRACE_LIBS, ("notAfter attribute is missing, the rule is valid\n")); *result = true; ret = EOK; + goto done; + } else if (ret != EOK) { + goto done; } for (i=0; values[i] ; i++) { |