diff options
author | Jakub Hrozek <jhrozek@redhat.com> | 2013-06-23 17:39:50 +0200 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2013-06-27 16:59:05 +0200 |
commit | 2d9bcb6249d74f7a2bb259e5da39646ec8913955 (patch) | |
tree | 306dcf7d114a8d7332f40af4dc4af338766a891d | |
parent | a0cf92bf59e02c5b5889d2b86a8ad98e75ee211a (diff) | |
download | sssd-2d9bcb6249d74f7a2bb259e5da39646ec8913955.tar.gz sssd-2d9bcb6249d74f7a2bb259e5da39646ec8913955.tar.xz sssd-2d9bcb6249d74f7a2bb259e5da39646ec8913955.zip |
IPA: Do not download or store the member attribute of host groupssssd-1.5.1-70
https://fedorahosted.org/sssd/ticket/1806
The IPA provider attempted to store the original value of member attribute
to the cache. That caused the memberof plugin to process the values which
was really CPU intensive.
-rw-r--r-- | src/providers/ipa/ipa_hbac_hosts.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/src/providers/ipa/ipa_hbac_hosts.c b/src/providers/ipa/ipa_hbac_hosts.c index 667cf9066..1fd5aa2f1 100644 --- a/src/providers/ipa/ipa_hbac_hosts.c +++ b/src/providers/ipa/ipa_hbac_hosts.c @@ -106,7 +106,7 @@ ipa_hbac_host_info_send(TALLOC_CTX *mem_ctx, goto immediate; } - state->attrs = talloc_array(state, const char *, 8); + state->attrs = talloc_array(state, const char *, 7); if (state->attrs == NULL) { DEBUG(1, ("Failed to allocate host attribute list.\n")); ret = ENOMEM; @@ -116,10 +116,9 @@ ipa_hbac_host_info_send(TALLOC_CTX *mem_ctx, state->attrs[1] = IPA_HOST_SERVERHOSTNAME; state->attrs[2] = IPA_HOST_FQDN; state->attrs[3] = IPA_UNIQUE_ID; - state->attrs[4] = IPA_MEMBER; - state->attrs[5] = IPA_MEMBEROF; - state->attrs[6] = IPA_CN; - state->attrs[7] = NULL; + state->attrs[4] = IPA_MEMBEROF; + state->attrs[5] = IPA_CN; + state->attrs[6] = NULL; subreq = sdap_get_generic_send(state, ev, opts, sh, search_base, LDAP_SCOPE_SUB, host_filter, |