diff options
| author | Michal Zidek <mzidek@redhat.com> | 2012-11-14 15:36:22 +0100 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2012-11-28 14:45:44 +0100 |
| commit | 3f964af2d45b85c8c7461f5227b343340ab06b06 (patch) | |
| tree | 05005911a805f91ceb730e3221bbf7211b1e111f /src/lib/idmap/sss_idmap.c | |
| parent | fac2e9f6f7551bf5bce89605cdb0672871613ca6 (diff) | |
| download | sssd-3f964af2d45b85c8c7461f5227b343340ab06b06.tar.gz sssd-3f964af2d45b85c8c7461f5227b343340ab06b06.tar.xz sssd-3f964af2d45b85c8c7461f5227b343340ab06b06.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] == '-' && |