diff options
| author | Stephen Gallagher <sgallagh@redhat.com> | 2010-06-09 09:54:11 -0400 |
|---|---|---|
| committer | Stephen Gallagher <sgallagh@redhat.com> | 2010-06-10 10:17:38 -0400 |
| commit | 6d8973d78dcf83c943f3cbc0cdb509987994a01b (patch) | |
| tree | bf1e64a5f3c9934f6053736949d09e4a61943aef /src/providers | |
| parent | f37bf8657971d16a4849f58242319699dca2de95 (diff) | |
| download | sssd-6d8973d78dcf83c943f3cbc0cdb509987994a01b.tar.gz sssd-6d8973d78dcf83c943f3cbc0cdb509987994a01b.tar.xz sssd-6d8973d78dcf83c943f3cbc0cdb509987994a01b.zip | |
Properly handle missing originalMemberOf entry in initgroups
Failing to return after the tevent_req_post() here can result in a
null-pointer dereference (along with other hard-to-track bugs)
https://fedorahosted.org/sssd/ticket/507
Diffstat (limited to 'src/providers')
| -rw-r--r-- | src/providers/ldap/sdap_async_accounts.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/providers/ldap/sdap_async_accounts.c b/src/providers/ldap/sdap_async_accounts.c index f949e46a0..46c622d4e 100644 --- a/src/providers/ldap/sdap_async_accounts.c +++ b/src/providers/ldap/sdap_async_accounts.c @@ -1181,6 +1181,7 @@ static struct tevent_req *sdap_initgr_nested_send(TALLOC_CTX *memctx, /* user with no groups ? */ tevent_req_error(req, ENOENT); tevent_req_post(req, ev); + return req; } state->count = el->num_values; |