214static void sdap_sudo_refresh_timeout(struct tevent_context *ev,
215                                      struct tevent_timer *te,
216                                      struct timeval tv, void *pvt)
217{
218    struct tevent_req *req = talloc_get_type(pvt, struct tevent_req);
219    struct sdap_sudo_refresh_ctx *refresh_ctx = NULL;
220    int delay;
221
222    refresh_ctx = tevent_req_callback_data(req, struct sdap_sudo_refresh_ctx);
223
224    delay = dp_opt_get_int(refresh_ctx->opts->basic, SDAP_SUDO_REFRESH_TIMEOUT);
At conditional (1): "debug_level & __debug_macro_newlevel" taking the true branch.
At conditional (2): "debug_timestamps" taking the true branch.
At conditional (3): "debug_microseconds" taking the true branch.
225    DEBUG(SSSDBG_CRIT_FAILURE, ("Refreshing SUDO rules timed out!"
226          " Timeout too small? (%ds)!\n", delay));
227
228    tv = tevent_timeval_current_ofs(delay, 0);
CID 12569: Unchecked return value (CHECKED_RETURN)Calling function "sdap_sudo_refresh_set_timer" without checking return value (as is done elsewhere 6 out of 7 times).
No check of the return value of "sdap_sudo_refresh_set_timer(refresh_ctx, tv)".
229    sdap_sudo_refresh_set_timer(refresh_ctx, tv);
230
231    talloc_zfree(req);
232}

207    tv = tevent_timeval_add(&tv, delay, 0);
Assigning: "ret" = return value from "sdap_sudo_refresh_set_timer(refresh_ctx, tv)".
208    ret = sdap_sudo_refresh_set_timer(refresh_ctx, tv);
"ret" has its value checked in "ret != 0".
209    if (ret != EOK) {
210        DEBUG(SSSDBG_CRIT_FAILURE, ("Error setting up SUDO refresh timer\n"));
211    }