Skip to content

Commit 51e9017

Browse files
committed
Fix GH-8517: FPM child can be possibly freed before stdio event
We should delete event before freeing child so it cannot be triggered after the child is freed.
1 parent 45e224c commit 51e9017

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

Diff for: sapi/fpm/fpm/fpm_children.c

+2
Original file line numberDiff line numberDiff line change
@@ -72,6 +72,7 @@ static void fpm_child_close(struct fpm_child_s *child, int in_event_loop) /* {{{
7272
if (child->fd_stdout != -1) {
7373
close(child->fd_stdout);
7474
}
75+
fpm_event_del(&child->ev_stdout);
7576
}
7677

7778
if (child->fd_stderr != -1) {
@@ -81,6 +82,7 @@ static void fpm_child_close(struct fpm_child_s *child, int in_event_loop) /* {{{
8182
if (child->fd_stderr != -1) {
8283
close(child->fd_stderr);
8384
}
85+
fpm_event_del(&child->ev_stderr);
8486
}
8587

8688
fpm_child_free(child);

0 commit comments

Comments
 (0)