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/responder/sudo/sudosrv_dp.c | |
| 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/responder/sudo/sudosrv_dp.c')
| -rw-r--r-- | src/responder/sudo/sudosrv_dp.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/responder/sudo/sudosrv_dp.c b/src/responder/sudo/sudosrv_dp.c index 95595fba2..3c3ac8b75 100644 --- a/src/responder/sudo/sudosrv_dp.c +++ b/src/responder/sudo/sudosrv_dp.c @@ -62,8 +62,7 @@ sss_dp_get_sudoers_send(TALLOC_CTX *mem_ctx, req = tevent_req_create(mem_ctx, &state, struct sss_dp_req_state); if (!req) { - ret = ENOMEM; - goto error; + return NULL; } if (!dom) { |