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/resolv/async_resolv.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/resolv/async_resolv.c')
| -rw-r--r-- | src/resolv/async_resolv.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/resolv/async_resolv.c b/src/resolv/async_resolv.c index 103592420..58d5c6e55 100644 --- a/src/resolv/async_resolv.c +++ b/src/resolv/async_resolv.c @@ -694,8 +694,7 @@ resolv_gethostbyname_files_send(TALLOC_CTX *mem_ctx, req = tevent_req_create(mem_ctx, &state, struct gethostbyname_files_state); if (req == NULL) { - tevent_req_error(req, ENOMEM); - goto done; + return NULL; } state->resolv_ctx = ctx; |