summaryrefslogtreecommitdiffstats
path: root/src/db
diff options
context:
space:
mode:
authorJakub Hrozek <jhrozek@redhat.com>2014-08-16 19:41:40 +0200
committerJakub Hrozek <jhrozek@redhat.com>2014-08-19 15:07:14 +0200
commitff541559e911b91b161f1b8411dd9d8cc60f2a18 (patch)
tree81e24a544f29aca475ce12d78166b3c570a03b8c /src/db
parent25dfb62595f73d1ca3d1170abe5853c4253d7c42 (diff)
downloadsssd-ff541559e911b91b161f1b8411dd9d8cc60f2a18.tar.gz
sssd-ff541559e911b91b161f1b8411dd9d8cc60f2a18.tar.xz
sssd-ff541559e911b91b161f1b8411dd9d8cc60f2a18.zip
SYSDB: Clarify sss_ldb_modify_permissive returns ldb error code
Reviewed-by: Lukáš Slebodník <lslebodn@redhat.com>
Diffstat (limited to 'src/db')
-rw-r--r--src/db/sysdb_ops.c6
1 files changed, 6 insertions, 0 deletions
diff --git a/src/db/sysdb_ops.c b/src/db/sysdb_ops.c
index 252097f28..e32d79a70 100644
--- a/src/db/sysdb_ops.c
+++ b/src/db/sysdb_ops.c
@@ -129,6 +129,9 @@ done:
* The wrapper around ldb_modify that uses LDB_CONTROL_PERMISSIVE_MODIFY_OID
* so that on adds entries that already exist are skipped and similarly
* entries that are missing are ignored on deletes
+ *
+ * Please note this function returns LDB error codes, not sysdb error
+ * codes on purpose, see usage in callers!
*/
int sss_ldb_modify_permissive(struct ldb_context *ldb,
struct ldb_message *msg)
@@ -159,6 +162,9 @@ int sss_ldb_modify_permissive(struct ldb_context *ldb,
talloc_free(req);
+ /* Please note this function returns LDB error codes, not sysdb error
+ * codes on purpose, see usage in callers!
+ */
return ret;
}