On Mon, Sep 01, 2014 at 10:37:53PM +0200, Daniel Gollub wrote:
In the uid=0 case (to obtain new free id) only uidNumber and
gidNumber
attributes got written, but not the additonal provided attributes like
alias or others.
---
src/db/sysdb_ops.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/db/sysdb_ops.c b/src/db/sysdb_ops.c
index e32d79a..b1a2992 100644
--- a/src/db/sysdb_ops.c
+++ b/src/db/sysdb_ops.c
@@ -1401,7 +1401,8 @@ int sysdb_add_user(struct sss_domain_info *domain,
}
ret = sysdb_set_user_attr(domain, name, id_attrs, SYSDB_MOD_REP);
- goto done;
+ /* continue on success, to commit additional attrs */
+ if (ret) goto done;
}
if (!attrs) {
--
1.9.1
Pushed to master as: 23600a657f84bbd71dca4dd77d65a1c6f4f4ff86
Daniel, does the unit test I sent look good to you? If so, I'll push it
atop your patch so we never regress in this functionality..