diff options
author | Simo Sorce <ssorce@redhat.com> | 2009-12-10 14:53:17 -0500 |
---|---|---|
committer | Stephen Gallagher <sgallagh@redhat.com> | 2009-12-10 17:20:19 -0500 |
commit | 51a52431ed52e0312c680e52371882088482071f (patch) | |
tree | ca2a20e9449110742c17a07e2f027201b376efdd /server | |
parent | 56c20b800541856559319363c6a3ae465a86d550 (diff) | |
download | sssd-51a52431ed52e0312c680e52371882088482071f.tar.gz sssd-51a52431ed52e0312c680e52371882088482071f.tar.xz sssd-51a52431ed52e0312c680e52371882088482071f.zip |
Fix for #316
We were never filling the group attrs because of an inverse return check.
Plus fix a crash bug for using a pointer that is not a memory context.
Diffstat (limited to 'server')
-rw-r--r-- | server/db/sysdb.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/server/db/sysdb.c b/server/db/sysdb.c index a87b5c565..10da9750f 100644 --- a/server/db/sysdb.c +++ b/server/db/sysdb.c @@ -271,7 +271,7 @@ int sysdb_attrs_users_from_str_list(struct sysdb_attrs *attrs, int ret; ret = sysdb_attrs_get_el(attrs, attr_name, &el); - if (!ret) { + if (ret) { return ret; } @@ -318,11 +318,11 @@ int sysdb_attrs_users_from_ldb_vals(struct sysdb_attrs *attrs, int ret; ret = sysdb_attrs_get_el(attrs, attr_name, &el); - if (!ret) { + if (ret) { return ret; } - vals = talloc_realloc(el, el->values, struct ldb_val, + vals = talloc_realloc(attrs->a, el->values, struct ldb_val, el->num_values + num_values); if (!vals) { return ENOMEM; |