summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSumit Bose <sbose@redhat.com>2015-04-22 15:10:07 +0200
committerJakub Hrozek <jhrozek@redhat.com>2015-04-27 15:39:12 +0200
commit2ab9a4538eb2e1a255e645f7efdcfd6bb722d265 (patch)
treedbfcc15493ae939f63ca13fddfbf413c69838c9e
parentbbd6f73bbad478a450ecfa2933a63de6dc269778 (diff)
downloadsssd-2ab9a4538eb2e1a255e645f7efdcfd6bb722d265.tar.gz
sssd-2ab9a4538eb2e1a255e645f7efdcfd6bb722d265.tar.xz
sssd-2ab9a4538eb2e1a255e645f7efdcfd6bb722d265.zip
IPA: do not try to save override data for the default view
For the default view all override data is available in the cached user or group object. Even if separate override data is available it should not be written into the cache. Resolves https://fedorahosted.org/sssd/ticket/2630 Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
-rw-r--r--src/providers/ipa/ipa_s2n_exop.c15
1 files changed, 10 insertions, 5 deletions
diff --git a/src/providers/ipa/ipa_s2n_exop.c b/src/providers/ipa/ipa_s2n_exop.c
index a844ee99d..b9de88984 100644
--- a/src/providers/ipa/ipa_s2n_exop.c
+++ b/src/providers/ipa/ipa_s2n_exop.c
@@ -2164,11 +2164,16 @@ static errno_t ipa_s2n_save_objects(struct sss_domain_info *dom,
goto done;
}
- ret = sysdb_store_override(dom, view_name, type, override_attrs,
- res->msgs[0]->dn);
- if (ret != EOK) {
- DEBUG(SSSDBG_OP_FAILURE, "sysdb_store_override failed.\n");
- goto done;
+ if (strcmp(view_name, SYSDB_DEFAULT_VIEW_NAME) != 0) {
+ /* For the default view the data return by the extdom plugin already
+ * contains all needed data and it is not expected to have a separate
+ * override object. */
+ ret = sysdb_store_override(dom, view_name, type, override_attrs,
+ res->msgs[0]->dn);
+ if (ret != EOK) {
+ DEBUG(SSSDBG_OP_FAILURE, "sysdb_store_override failed.\n");
+ goto done;
+ }
}
done: