diff options
| author | Lukas Slebodnik <lslebodn@redhat.com> | 2016-07-25 08:31:17 +0200 |
|---|---|---|
| committer | Lukas Slebodnik <lslebodn@redhat.com> | 2016-07-25 15:49:27 +0200 |
| commit | a2eba280a64dc40a0735ecc0b8e2548ad0b5aa1a (patch) | |
| tree | 3a357a1db74c444f207413421e78cfd2d0eefd9b /src/db | |
| parent | bc7991db97482eb2ac77f7105ee4bb3d329acff7 (diff) | |
| download | sssd-a2eba280a64dc40a0735ecc0b8e2548ad0b5aa1a.tar.gz sssd-a2eba280a64dc40a0735ecc0b8e2548ad0b5aa1a.tar.xz sssd-a2eba280a64dc40a0735ecc0b8e2548ad0b5aa1a.zip | |
SYSDB: Do not try to modify ts cache for unsupported DNs
Only users and groups have timestamp data in separate cache.
It caused false positive warnings for autofs, netgroup ...
Reviewed-by: Petr Čech <pcech@redhat.com>
Diffstat (limited to 'src/db')
| -rw-r--r-- | src/db/sysdb_ops.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/src/db/sysdb_ops.c b/src/db/sysdb_ops.c index 4755ea342..19d6be03e 100644 --- a/src/db/sysdb_ops.c +++ b/src/db/sysdb_ops.c @@ -1198,9 +1198,14 @@ int sysdb_set_entry_attr(struct sysdb_ctx *sysdb, sysdb_write = sysdb_entry_attrs_diff(sysdb, entry_dn, attrs, mod_op); if (sysdb_write == true) { ret = sysdb_set_cache_entry_attr(sysdb->ldb, entry_dn, attrs, mod_op); + if (ret != EOK) { + DEBUG(SSSDBG_MINOR_FAILURE, + "Cannot set attrs for %s, %d [%s]\n", + ldb_dn_get_linearized(entry_dn), ret, sss_strerror(ret)); + } } - if (ret == EOK) { + if (ret == EOK && is_ts_ldb_dn(entry_dn)) { tret = sysdb_set_ts_entry_attr(sysdb, entry_dn, attrs, mod_op); if (tret != EOK) { DEBUG(SSSDBG_MINOR_FAILURE, |