diff options
| author | Lukas Slebodnik <lslebodn@redhat.com> | 2014-10-10 19:29:38 +0200 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2014-10-12 17:25:40 +0200 |
| commit | 6af2095e63f5f78804420fb61e66cb741c179b5d (patch) | |
| tree | ad949cf9c9716f95f780b140c9c0cf5dfe1e1f11 | |
| parent | 39c6d3d8e583e45d2b646b75e479a5e97f749156 (diff) | |
| download | sssd-6af2095e63f5f78804420fb61e66cb741c179b5d.tar.gz sssd-6af2095e63f5f78804420fb61e66cb741c179b5d.tar.xz sssd-6af2095e63f5f78804420fb61e66cb741c179b5d.zip | |
SDAP: Fix warning Value stored to is never read
If function sdap_idmap_sid_to_unix return error code ENOTSUP
then error message will be printed: "Skipping buit-in object"
Processing of sids will continue an variable ret will be overridden with return
code from function sdap_idmap_sid_to_unix, therefore storing EOK to variable
is useless.
Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
| -rw-r--r-- | src/providers/ldap/sdap_async_initgroups_ad.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/providers/ldap/sdap_async_initgroups_ad.c b/src/providers/ldap/sdap_async_initgroups_ad.c index 9923958b8..a533279f6 100644 --- a/src/providers/ldap/sdap_async_initgroups_ad.c +++ b/src/providers/ldap/sdap_async_initgroups_ad.c @@ -897,7 +897,6 @@ static void sdap_ad_tokengroups_initgr_mapping_done(struct tevent_req *subreq) ret = sdap_idmap_sid_to_unix(state->idmap_ctx, sid, &gid); if (ret == ENOTSUP) { DEBUG(SSSDBG_TRACE_FUNC, "Skipping built-in object.\n"); - ret = EOK; continue; } else if (ret != EOK) { DEBUG(SSSDBG_MINOR_FAILURE, "Could not convert SID to GID: [%s]. " |