diff options
| author | Pavel Březina <pbrezina@redhat.com> | 2016-01-13 13:15:09 +0100 |
|---|---|---|
| committer | Lukas Slebodnik <lslebodn@redhat.com> | 2016-01-15 10:02:14 +0100 |
| commit | 468495d91d536603a1c485424275b6dcf2bb83de (patch) | |
| tree | 92864b8f8c96ad4892f00283fe7cc7fff7f39a18 | |
| parent | 383840c0c9c440710352076f844a64745121d251 (diff) | |
| download | sssd-468495d91d536603a1c485424275b6dcf2bb83de.tar.gz sssd-468495d91d536603a1c485424275b6dcf2bb83de.tar.xz sssd-468495d91d536603a1c485424275b6dcf2bb83de.zip | |
SDAP: do not fail if refs are found but not processed
It is possible to end up with not-processed referrals when
using AD provider and ldap_referrals=true.
Resolves:
https://fedorahosted.org/sssd/ticket/2906
Reviewed-by: Stephen Gallagher <sgallagh@redhat.com>
| -rw-r--r-- | src/providers/ldap/sdap_async.c | 11 |
1 files changed, 1 insertions, 10 deletions
diff --git a/src/providers/ldap/sdap_async.c b/src/providers/ldap/sdap_async.c index 668bd7b46..5260aafeb 100644 --- a/src/providers/ldap/sdap_async.c +++ b/src/providers/ldap/sdap_async.c @@ -1653,16 +1653,6 @@ static void generic_ext_search_handler(struct tevent_req *subreq, } if (ref_count > 0) { - if (dp_opt_get_bool(opts->basic, SDAP_REFERRALS)) { - /* We got back referrals here, but they should have - * been processed internally by openldap libs. - * This should never happen. - */ - talloc_free(refs); - tevent_req_error(req, EINVAL); - return; - } - /* We will ignore referrals in the generic handler */ DEBUG(SSSDBG_TRACE_ALL, "Request included referrals which were ignored.\n"); @@ -1674,6 +1664,7 @@ static void generic_ext_search_handler(struct tevent_req *subreq, } } + talloc_free(refs); tevent_req_done(req); } |