-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
Hi, I found these issues while testing Stephen's initgroup patches:
[PATCH 1/3] Fix sysdb_group_dn_name We copied the RDN attribute name, not its value. Also includes a unit test.
[PATCH 2/3] Fix sysdb_attrs_to_list We didn't cycle through all the attributes.
[PATCH 3/3] Request the correct attribute name The attributes we got from sdap_get_generic_recv() are already mapped to sysdb attributes, so when we asked for "cn", it was not found as the attributes already included "name".
They apply to master only, I can resend the 1.2 version when/if these are deemed OK.
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
On 09/16/2010 11:46 AM, Jakub Hrozek wrote:
Hi, I found these issues while testing Stephen's initgroup patches:
[PATCH 1/3] Fix sysdb_group_dn_name We copied the RDN attribute name, not its value. Also includes a unit test.
[PATCH 2/3] Fix sysdb_attrs_to_list We didn't cycle through all the attributes.
[PATCH 3/3] Request the correct attribute name The attributes we got from sdap_get_generic_recv() are already mapped to sysdb attributes, so when we asked for "cn", it was not found as the attributes already included "name".
They apply to master only, I can resend the 1.2 version when/if these are deemed OK.
Ack.
Please send out 1.2 versions as well.
- -- Stephen Gallagher RHCE 804006346421761
Delivering value year after year. Red Hat ranks #1 in value among software vendors. http://www.redhat.com/promo/vendor/
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
On 09/22/2010 09:44 AM, Stephen Gallagher wrote:
On 09/16/2010 11:46 AM, Jakub Hrozek wrote:
Hi, I found these issues while testing Stephen's initgroup patches:
[PATCH 1/3] Fix sysdb_group_dn_name We copied the RDN attribute name, not its value. Also includes a unit test.
[PATCH 2/3] Fix sysdb_attrs_to_list We didn't cycle through all the attributes.
[PATCH 3/3] Request the correct attribute name The attributes we got from sdap_get_generic_recv() are already mapped to sysdb attributes, so when we asked for "cn", it was not found as the attributes already included "name".
They apply to master only, I can resend the 1.2 version when/if these are deemed OK.
Ack.
Please send out 1.2 versions as well.
Pushed to master.
- -- Stephen Gallagher RHCE 804006346421761
Delivering value year after year. Red Hat ranks #1 in value among software vendors. http://www.redhat.com/promo/vendor/
sssd-devel@lists.fedorahosted.org
-
Jakub Hrozek -
Stephen Gallagher