diff options
| author | Lukas Slebodnik <lslebodn@redhat.com> | 2016-06-16 17:40:50 +0200 |
|---|---|---|
| committer | Lukas Slebodnik <lslebodn@redhat.com> | 2016-06-29 12:18:57 +0200 |
| commit | b54cd7caf6b73b36b68fcefc0cad39a626690398 (patch) | |
| tree | 54e2e842cde33274d15b6554b144282ba33fa005 /src/responder | |
| parent | bdadcaf271818e88e56e86c2bd90663a08fd9721 (diff) | |
| download | sssd-b54cd7caf6b73b36b68fcefc0cad39a626690398.tar.gz sssd-b54cd7caf6b73b36b68fcefc0cad39a626690398.tar.xz sssd-b54cd7caf6b73b36b68fcefc0cad39a626690398.zip | |
Downcast to errno_t after tevent_req_is_error
Functions tevent_req_is_error and _tevent_req_error
use type uint64_t for error code.
SSSD uses errno_t which is an alias for int.
Therefore complier assumes that macro TEVENT_REQ_RETURN_ON_ERROR
can return 0 due to implicit down casting from uint64_t -> int.
This patch makes down casting explicit and returns EINVAL
if result of downcasting is 0.
Reviewed-by: Pavel Březina <pbrezina@redhat.com>
Diffstat (limited to 'src/responder')
| -rw-r--r-- | src/responder/common/responder_dp.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/src/responder/common/responder_dp.c b/src/responder/common/responder_dp.c index f53c7d232..fed47a068 100644 --- a/src/responder/common/responder_dp.c +++ b/src/responder/common/responder_dp.c @@ -397,14 +397,20 @@ sss_dp_req_recv(TALLOC_CTX *mem_ctx, tevent_req_data(sidereq, struct sss_dp_req_state); enum tevent_req_state TRROEstate; - uint64_t TRROEerr; + uint64_t TRROEuint64; + errno_t TRROEerr; *dp_err = state->dp_err; *dp_ret = state->dp_ret; *err_msg = talloc_steal(mem_ctx, state->err_msg); - if (tevent_req_is_error(sidereq, &TRROEstate, &TRROEerr)) { + if (tevent_req_is_error(sidereq, &TRROEstate, &TRROEuint64)) { + TRROEerr = (errno_t)TRROEuint64; + if (TRROEstate == TEVENT_REQ_USER_ERROR) { + if (TRROEerr == 0) { + return ERR_INTERNAL; + } *dp_err = DP_ERR_FATAL; *dp_ret = TRROEerr; } else { |