From 6d8973d78dcf83c943f3cbc0cdb509987994a01b Mon Sep 17 00:00:00 2001
From: Stephen Gallagher <sgallagh@redhat.com>
Date: Wed, 9 Jun 2010 09:54:11 -0400
Subject: 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
---
 src/providers/ldap/sdap_async_accounts.c | 1 +
 1 file changed, 1 insertion(+)

(limited to 'src/providers')

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;
 
-- 
cgit