Skip to content

Commit b5e8a61

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 b5e8a61

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

sapi/fpm/fpm/fpm_children.c

+2
Original file line numberDiff line numberDiff line change
@@ -68,6 +68,7 @@ static void fpm_child_close(struct fpm_child_s *child, int in_event_loop) /* {{{
6868
if (child->fd_stdout != -1) {
6969
if (in_event_loop) {
7070
fpm_event_fire(&child->ev_stdout);
71+
fpm_event_del(&child->ev_stdout);
7172
}
7273
if (child->fd_stdout != -1) {
7374
close(child->fd_stdout);
@@ -77,6 +78,7 @@ static void fpm_child_close(struct fpm_child_s *child, int in_event_loop) /* {{{
7778
if (child->fd_stderr != -1) {
7879
if (in_event_loop) {
7980
fpm_event_fire(&child->ev_stderr);
81+
fpm_event_del(&child->ev_stdout);
8082
}
8183
if (child->fd_stderr != -1) {
8284
close(child->fd_stderr);

0 commit comments

Comments
 (0)