diff options
author | Stephen Gallagher <sgallagh@redhat.com> | 2010-05-06 11:23:23 -0400 |
---|---|---|
committer | Stephen Gallagher <sgallagh@redhat.com> | 2010-05-07 17:14:31 -0400 |
commit | 9e99e3c5c45b20189e76e4d2546966ff1fb3ce40 (patch) | |
tree | 237e8be9cbfdfc3257c6749b4033be208b1133aa /src/providers/ldap/ldap_id_enum.c | |
parent | d293186e98a9bd68c8961980978485adae082788 (diff) | |
download | sssd-9e99e3c5c45b20189e76e4d2546966ff1fb3ce40.tar.gz sssd-9e99e3c5c45b20189e76e4d2546966ff1fb3ce40.tar.bz2 sssd-9e99e3c5c45b20189e76e4d2546966ff1fb3ce40.zip |
Fix segfault in GSSAPI reconnect code
Also clean up some duplicated code into a single common routine
sdap_account_info_common_done()
Diffstat (limited to 'src/providers/ldap/ldap_id_enum.c')
-rw-r--r-- | src/providers/ldap/ldap_id_enum.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/providers/ldap/ldap_id_enum.c b/src/providers/ldap/ldap_id_enum.c index d86b38d0..89ca2de7 100644 --- a/src/providers/ldap/ldap_id_enum.c +++ b/src/providers/ldap/ldap_id_enum.c @@ -234,7 +234,9 @@ fail: (int)err, strerror(err))); if (sdap_check_gssapi_reconnect(state->ctx)) { - state->ctx->gsh->connected = false; + if (state->ctx->gsh) { + state->ctx->gsh->connected = false; + } ret = ldap_id_enum_users_restart(req); if (ret == EOK) return; } @@ -282,7 +284,9 @@ static void ldap_id_enum_groups_done(struct tevent_req *subreq) fail: /* check if credentials are expired otherwise go offline on failures */ if (sdap_check_gssapi_reconnect(state->ctx)) { - state->ctx->gsh->connected = false; + if (state->ctx->gsh) { + state->ctx->gsh->connected = false; + } ret = ldap_id_enum_groups_restart(req); if (ret == EOK) return; } |