diff options
| author | Pavel Reichl <preichl@redhat.com> | 2014-08-01 11:13:20 +0100 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2014-08-19 17:50:48 +0200 |
| commit | 310793b257ec4b25d91a5300dc664a07b2c863f8 (patch) | |
| tree | cb92903ed8c619eba03b85a6769b8304a9f5af1d | |
| parent | 1a202de64cd5e0ea8817dc77c4e3b669976d8482 (diff) | |
| download | sssd-310793b257ec4b25d91a5300dc664a07b2c863f8.tar.gz sssd-310793b257ec4b25d91a5300dc664a07b2c863f8.tar.xz sssd-310793b257ec4b25d91a5300dc664a07b2c863f8.zip | |
SDAP: nitpicks in sdap_access_filter_get_access_done
Fixed typo and replaced duplicated string by macro definition.
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
| -rw-r--r-- | src/providers/ldap/sdap_access.c | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/src/providers/ldap/sdap_access.c b/src/providers/ldap/sdap_access.c index f5900674e..1143f0156 100644 --- a/src/providers/ldap/sdap_access.c +++ b/src/providers/ldap/sdap_access.c @@ -40,6 +40,8 @@ #include "providers/data_provider.h" #include "providers/dp_backend.h" +#define MALFORMED_FILTER "Malformed access control filter [%s]\n" + static errno_t sdap_save_user_cache_bool(struct sss_domain_info *domain, const char *username, const char *attr_name, @@ -875,10 +877,8 @@ static void sdap_access_filter_get_access_done(struct tevent_req *subreq) } else if (dp_error == DP_ERR_OFFLINE) { ret = sdap_access_filter_decide_offline(req); } else if (ret == ERR_INVALID_FILTER) { - sss_log(SSS_LOG_ERR, - "Malformed access control filter [%s]\n", state->filter); - DEBUG(SSSDBG_CRIT_FAILURE, - "Malformed access control filter [%s]\n", state->filter); + sss_log(SSS_LOG_ERR, MALFORMED_FILTER, state->filter); + DEBUG(SSSDBG_CRIT_FAILURE, MALFORMED_FILTER, state->filter); ret = ERR_ACCESS_DENIED; } else { DEBUG(SSSDBG_CRIT_FAILURE, @@ -918,9 +918,7 @@ static void sdap_access_filter_get_access_done(struct tevent_req *subreq) } if (found) { - /* Save "allow" to the cache for future offline - :q* access checks. - */ + /* Save "allow" to the cache for future offline access checks. */ DEBUG(SSSDBG_TRACE_FUNC, "Access granted by online lookup\n"); ret = EOK; } |