Exited child should still definitely be in the parent state

diff --git a/src/lib/unlang/parallel.c b/src/lib/unlang/parallel.c
index e7793775f1965c68ca23af589056d694dc9d68c8..8e06c2ed7a0c11f7495d96965114652dc487f72e 100644 (file)
--- a/src/lib/unlang/parallel.c
+++ b/src/lib/unlang/parallel.c
@@ -42,9 +42,13 @@ static inline CC_HINT(always_inline) void unlang_parallel_cancel_child(unlang_pa
 
        switch (state->children[i].state) {
        case CHILD_INIT:
-       case CHILD_EXITED:
+               state->children[i].state = CHILD_CANCELLED;
                fr_assert(!state->children[i].request);
+               break;
+
+       case CHILD_EXITED:
                state->children[i].state = CHILD_CANCELLED;
+               TALLOC_FREE(state->children[i].request);
                break;
 
        case CHILD_RUNNABLE:    /* Don't check runnable_id, may be yielded */