diff options
| author | Michal Zidek <mzidek@redhat.com> | 2012-11-14 15:36:22 +0100 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2012-11-28 11:20:09 +0100 |
| commit | d6f283302268520c1506fb3da4f2a22f5a741be5 (patch) | |
| tree | 1db58d6866fe821754a7e49d5442c2b427cdc075 /src/lib/idmap/sss_idmap.c | |
| parent | 82505163d22f0ce9cc63f22b2cac5d3ca3af0937 (diff) | |
| download | sssd-d6f283302268520c1506fb3da4f2a22f5a741be5.tar.gz sssd-d6f283302268520c1506fb3da4f2a22f5a741be5.tar.xz sssd-d6f283302268520c1506fb3da4f2a22f5a741be5.zip | |
idmap: Silence DEBUG messages when dealing with built-in SIDs.
When converting built-in SID to unix GID/UID a confusing debug
message about the failed conversion was printed. This patch special
cases these built-in objects.
https://fedorahosted.org/sssd/ticket/1593
Diffstat (limited to 'src/lib/idmap/sss_idmap.c')
| -rw-r--r-- | src/lib/idmap/sss_idmap.c | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/src/lib/idmap/sss_idmap.c b/src/lib/idmap/sss_idmap.c index c589bd458..d7254e3ea 100644 --- a/src/lib/idmap/sss_idmap.c +++ b/src/lib/idmap/sss_idmap.c @@ -280,6 +280,15 @@ fail: return IDMAP_OUT_OF_MEMORY; } +static bool sss_idmap_sid_is_builtin(const char *sid) +{ + if (strncmp(sid, "S-1-5-32-", 9) == 0) { + return true; + } + + return false; +} + enum idmap_error_code sss_idmap_sid_to_unix(struct sss_idmap_ctx *ctx, const char *sid, uint32_t *id) @@ -293,6 +302,10 @@ enum idmap_error_code sss_idmap_sid_to_unix(struct sss_idmap_ctx *ctx, idmap_domain_info = ctx->idmap_domain_info; + if (sid && sss_idmap_sid_is_builtin(sid)) { + return IDMAP_BUILTIN_SID; + } + while (idmap_domain_info != NULL) { dom_len = strlen(idmap_domain_info->sid); if (strlen(sid) > dom_len && sid[dom_len] == '-' && |