Skip to content

Commit

Permalink
check return for CID 1544245
Browse files Browse the repository at this point in the history
Realistically, unlang_xlat_yield() _always_ returns
XLAT_ACTION_YIELD, and the only reason it returns anything
is so that we can do "return unlang_xlat_yield()"
  • Loading branch information
alandekok committed Sep 6, 2023
1 parent 16d1f50 commit f07c556
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/lib/unlang/xlat_eval.c
Original file line number Diff line number Diff line change
Expand Up @@ -858,7 +858,7 @@ xlat_action_t xlat_frame_eval_resume(TALLOC_CTX *ctx, fr_dcursor_t *out,
return xa;

case XLAT_ACTION_DONE:
unlang_xlat_yield(request, xlat_null_resume, NULL, 0, NULL);
if (unlang_xlat_yield(request, xlat_null_resume, NULL, 0, NULL) != XLAT_ACTION_YIELD) return XLAT_ACTION_FAIL;

fr_dcursor_next(out); /* Wind to the start of this functions output */
RDEBUG2("| --> %pV", fr_dcursor_current(out));
Expand Down

0 comments on commit f07c556

Please sign in to comment.