summaryrefslogtreecommitdiffstats
path: root/src/providers/ipa
diff options
context:
space:
mode:
authorSumit Bose <sbose@redhat.com>2014-12-05 11:03:48 +0100
committerJakub Hrozek <jhrozek@redhat.com>2015-01-14 12:00:33 +0100
commit2fc12875f7d51248799016c19c1298b85e06a286 (patch)
tree2f0ec11f59aa2bec28cbbecb8779b51f420dc4cf /src/providers/ipa
parentd8ceb194023a2cdc8bc183acc322e9a7fb6fe2b1 (diff)
downloadsssd-2fc12875f7d51248799016c19c1298b85e06a286.tar.gz
sssd-2fc12875f7d51248799016c19c1298b85e06a286.tar.xz
sssd-2fc12875f7d51248799016c19c1298b85e06a286.zip
IPA: make version check more precise
The call protected by the check does not only expect the version 1 of the extdom plugin is used but a specific response type as well. Since version 1 can return older response types as well we want to be on the safe side. Reviewed-by: Jakub Hrozek <jhrozek@redhat.com>
Diffstat (limited to 'src/providers/ipa')
-rw-r--r--src/providers/ipa/ipa_s2n_exop.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/providers/ipa/ipa_s2n_exop.c b/src/providers/ipa/ipa_s2n_exop.c
index 1d9a79a38..b02387a82 100644
--- a/src/providers/ipa/ipa_s2n_exop.c
+++ b/src/providers/ipa/ipa_s2n_exop.c
@@ -759,7 +759,7 @@ static errno_t s2n_response_to_attrs(TALLOC_CTX *mem_ctx,
attrs->a.user.pw_uid = uid;
attrs->a.user.pw_gid = gid;
- if (is_v1) {
+ if (is_v1 && type == RESP_USER_GROUPLIST) {
ret = add_v1_user_data(ber, attrs);
if (ret != EOK) {
DEBUG(SSSDBG_OP_FAILURE, "add_v1_user_data failed.\n");
@@ -798,7 +798,7 @@ static errno_t s2n_response_to_attrs(TALLOC_CTX *mem_ctx,
attrs->a.group.gr_gid = gid;
- if (is_v1) {
+ if (is_v1 && type == RESP_GROUP_MEMBERS) {
ret = add_v1_group_data(ber, attrs);
if (ret != EOK) {
DEBUG(SSSDBG_OP_FAILURE, "add_v1_group_data failed.\n");