diff options
| author | Sumit Bose <sbose@redhat.com> | 2014-12-05 11:03:48 +0100 |
|---|---|---|
| committer | Jakub Hrozek <jhrozek@redhat.com> | 2015-01-14 12:00:33 +0100 |
| commit | 2fc12875f7d51248799016c19c1298b85e06a286 (patch) | |
| tree | 2f0ec11f59aa2bec28cbbecb8779b51f420dc4cf /src/providers/ipa | |
| parent | d8ceb194023a2cdc8bc183acc322e9a7fb6fe2b1 (diff) | |
| download | sssd-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.c | 4 |
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"); |