summaryrefslogtreecommitdiffstats
path: root/src/db/sysdb_ops.c
diff options
context:
space:
mode:
Diffstat (limited to 'src/db/sysdb_ops.c')
-rw-r--r--src/db/sysdb_ops.c93
1 files changed, 79 insertions, 14 deletions
diff --git a/src/db/sysdb_ops.c b/src/db/sysdb_ops.c
index d16cfb920..252097f28 100644
--- a/src/db/sysdb_ops.c
+++ b/src/db/sysdb_ops.c
@@ -1539,7 +1539,14 @@ int sysdb_add_group(struct sss_domain_info *domain,
ret = sysdb_search_user_by_name(tmp_ctx, domain, name, NULL, &msg);
if (ret != ENOENT) {
- if (ret == EOK) ret = EEXIST;
+ if (ret == EOK) {
+ DEBUG(SSSDBG_TRACE_LIBS, "MPG domain contains a user "
+ "with the same name - %s.\n", name);
+ ret = EEXIST;
+ } else {
+ DEBUG(SSSDBG_TRACE_LIBS,
+ "sysdb_search_user_by_name failed for user %s.\n", name);
+ }
goto done;
}
}
@@ -1548,18 +1555,32 @@ int sysdb_add_group(struct sss_domain_info *domain,
if (gid != 0) {
ret = sysdb_search_group_by_gid(tmp_ctx, domain, gid, NULL, &msg);
if (ret != ENOENT) {
- if (ret == EOK) ret = EEXIST;
+ if (ret == EOK) {
+ DEBUG(SSSDBG_TRACE_LIBS,
+ "Group with the same gid exists: [%"SPRIgid"].\n", gid);
+ ret = EEXIST;
+ } else {
+ DEBUG(SSSDBG_TRACE_LIBS,
+ "sysdb_search_group_by_gid failed for gid: "
+ "[%"SPRIgid"].\n", gid);
+ }
goto done;
}
}
/* try to add the group */
ret = sysdb_add_basic_group(domain, name, gid);
- if (ret) goto done;
+ if (ret) {
+ DEBUG(SSSDBG_TRACE_LIBS,
+ "sysdb_add_basic_group failed for: %s with gid: "
+ "[%"SPRIgid"].\n", name, gid);
+ goto done;
+ }
if (!attrs) {
attrs = sysdb_new_attrs(tmp_ctx);
if (!attrs) {
+ DEBUG(SSSDBG_TRACE_LIBS, "sysdb_new_attrs failed.\n");
ret = ENOMEM;
goto done;
}
@@ -1569,17 +1590,27 @@ int sysdb_add_group(struct sss_domain_info *domain,
if (ret == ENOENT) {
posix = true;
ret = sysdb_attrs_add_bool(attrs, SYSDB_POSIX, true);
- if (ret) goto done;
+ if (ret) {
+ DEBUG(SSSDBG_TRACE_LIBS, "Failed to add posix attribute.\n");
+ goto done;
+ }
} else if (ret != EOK) {
+ DEBUG(SSSDBG_TRACE_LIBS, "Failed to get posix attribute.\n");
goto done;
}
if (posix && gid == 0) {
ret = sysdb_get_new_id(domain, &id);
- if (ret) goto done;
+ if (ret) {
+ DEBUG(SSSDBG_TRACE_LIBS, "sysdb_get_new_id failed.\n");
+ goto done;
+ }
ret = sysdb_attrs_add_uint32(attrs, SYSDB_GIDNUM, id);
- if (ret) goto done;
+ if (ret) {
+ DEBUG(SSSDBG_TRACE_LIBS, "Failed to add new gid.\n");
+ goto done;
+ }
}
if (!now) {
@@ -1587,14 +1618,24 @@ int sysdb_add_group(struct sss_domain_info *domain,
}
ret = sysdb_attrs_add_time_t(attrs, SYSDB_LAST_UPDATE, now);
- if (ret) goto done;
+ if (ret) {
+ DEBUG(SSSDBG_TRACE_LIBS, "Failed to add sysdb-last-update.\n");
+ goto done;
+ }
ret = sysdb_attrs_add_time_t(attrs, SYSDB_CACHE_EXPIRE,
((cache_timeout) ?
(now + cache_timeout) : 0));
- if (ret) goto done;
+ if (ret) {
+ DEBUG(SSSDBG_TRACE_LIBS, "Failed to add sysdb-cache-expire.\n");
+ goto done;
+ }
ret = sysdb_set_group_attr(domain, name, attrs, SYSDB_MOD_REP);
+ if (ret) {
+ DEBUG(SSSDBG_TRACE_LIBS, "sysdb_set_group_attr failed.\n");
+ goto done;
+ }
done:
if (ret == EOK) {
@@ -2037,6 +2078,7 @@ int sysdb_store_group(struct sss_domain_info *domain,
goto done;
}
if (ret == ENOENT) {
+ DEBUG(SSSDBG_TRACE_LIBS, "Group %s does not exist.\n", name);
new_group = true;
}
@@ -2064,20 +2106,30 @@ int sysdb_store_group(struct sss_domain_info *domain,
/* This may be a group rename. If there is a group with the
* same GID, remove it and try to add the basic group again
*/
+ DEBUG(SSSDBG_TRACE_LIBS, "sysdb_add_group failed: [EEXIST].\n");
ret = sysdb_delete_group(domain, NULL, gid);
if (ret == ENOENT) {
/* Not found by GID, return the original EEXIST,
* this may be a conflict in MPG domain or something
* else */
+ DEBUG(SSSDBG_TRACE_LIBS,
+ "sysdb_delete_group failed (while renaming group). Not "
+ "found by gid: [%"SPRIgid"].\n", gid);
return EEXIST;
} else if (ret != EOK) {
+ DEBUG(SSSDBG_TRACE_LIBS, "sysdb_add_group failed.\n");
goto done;
}
DEBUG(SSSDBG_MINOR_FAILURE,
- "A group with the same GID [%llu] was removed from the "
- "cache\n", (unsigned long long) gid);
+ "A group with the same GID [%"SPRIgid"] was removed from "
+ "the cache\n", gid);
ret = sysdb_add_group(domain, name, gid, attrs, cache_timeout,
now);
+ if (ret) {
+ DEBUG(SSSDBG_MINOR_FAILURE,
+ "sysdb_add_group failed (while renaming group) for: "
+ "%s [%"SPRIgid"].\n", name, gid);
+ }
}
goto done;
}
@@ -2085,18 +2137,31 @@ int sysdb_store_group(struct sss_domain_info *domain,
/* the group exists, let's just replace attributes when set */
if (gid) {
ret = sysdb_attrs_add_uint32(attrs, SYSDB_GIDNUM, gid);
- if (ret) goto done;
+ if (ret) {
+ DEBUG(SSSDBG_TRACE_LIBS, "Failed to add GID.\n");
+ goto done;
+ }
}
ret = sysdb_attrs_add_time_t(attrs, SYSDB_LAST_UPDATE, now);
- if (ret) goto done;
+ if (ret) {
+ DEBUG(SSSDBG_TRACE_LIBS, "Failed to add sysdb-last-update.\n");
+ goto done;
+ }
ret = sysdb_attrs_add_time_t(attrs, SYSDB_CACHE_EXPIRE,
((cache_timeout) ?
(now + cache_timeout) : 0));
- if (ret) goto done;
+ if (ret) {
+ DEBUG(SSSDBG_TRACE_LIBS, "Failed to add sysdb-cache-expire.\n");
+ goto done;
+ }
ret = sysdb_set_group_attr(domain, name, attrs, SYSDB_MOD_REP);
+ if (ret) {
+ DEBUG(SSSDBG_TRACE_LIBS, "sysdb_set_group_attr failed.\n");
+ goto done;
+ }
done:
if (ret) {
@@ -3246,7 +3311,7 @@ int sysdb_cache_auth(struct sss_domain_info *domain,
SYSDB_LAST_FAILED_LOGIN,
time(NULL));
if (ret != EOK) {
- DEBUG(SSSDBG_MINOR_FAILURE, "sysdb_attrs_add_time_t failed\n.");
+ DEBUG(SSSDBG_MINOR_FAILURE, "sysdb_attrs_add_time_t failed.\n");
goto done;
}
generated by cgit (git 2.34.1) at 2024-05-05 02:18:34 +0000