diff options
| author | Sumit Bose <sbose@redhat.com> | 2016-02-10 15:15:41 +0100 |
|---|---|---|
| committer | Lukas Slebodnik <lslebodn@redhat.com> | 2016-02-11 14:39:12 +0100 |
| commit | a0c764a36f2f432e6063de84be6f6af7e96ec159 (patch) | |
| tree | 943cdaa2ed48cf66a54f9439702502145a7fce17 /src/providers | |
| parent | d862e55716e1b24168fd5deef44c4f9deb7c0709 (diff) | |
| download | sssd-a0c764a36f2f432e6063de84be6f6af7e96ec159.tar.gz sssd-a0c764a36f2f432e6063de84be6f6af7e96ec159.tar.xz sssd-a0c764a36f2f432e6063de84be6f6af7e96ec159.zip | |
Just return NULL if tevent_req_create() fails
In general we just return NULL if tevent_req_create() fails because
there is nothing we can do with the request anyway. Especially
tevent_req_error() should not be called because it tries to dereference
req.
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
Diffstat (limited to 'src/providers')
| -rw-r--r-- | src/providers/ldap/ldap_id.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/providers/ldap/ldap_id.c b/src/providers/ldap/ldap_id.c index a51c5271b..b7cef4e13 100644 --- a/src/providers/ldap/ldap_id.c +++ b/src/providers/ldap/ldap_id.c @@ -1490,8 +1490,7 @@ sdap_handle_acct_req_send(TALLOC_CTX *mem_ctx, req = tevent_req_create(mem_ctx, &state, struct sdap_handle_acct_req_state); if (!req) { - ret = ENOMEM; - goto done; + return NULL; } state->ar = ar; |