From a05ee9b926d398318e36944813a313bdff5eb999 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(+)

diff --git a/src/providers/ldap/sdap_async_accounts.c b/src/providers/ldap/sdap_async_accounts.c
index abebe2883..c5c3a5072 100644
--- a/src/providers/ldap/sdap_async_accounts.c
+++ b/src/providers/ldap/sdap_async_accounts.c
@@ -1660,6 +1660,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