Skip to content

Commit ff90d42

Browse files
devnexenbukka
authored andcommitted
Fix FPM kqueue handler fix remove callback typo
It avoids confusion on failed event.
1 parent 6186ecd commit ff90d42

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

NEWS

+1
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ PHP NEWS
1414
. Fixed bug #76003 (FPM /status reports wrong number of active processe).
1515
(Jakub Zelenka)
1616
. Fixed bug #77023 (FPM cannot shutdown processes). (Jakub Zelenka)
17+
. Fixed comment in kqueue remove callback log message. (David Carlier)
1718

1819
- MySQLi:
1920
. Fixed bug GH-8267 (MySQLi uses unsupported format specifier on Windows).

sapi/fpm/fpm/events/kqueue.c

+2-2
Original file line numberDiff line numberDiff line change
@@ -191,11 +191,11 @@ static int fpm_event_kqueue_remove(struct fpm_event_s *ev) /* {{{ */
191191
EV_SET(&k, ev->fd, EVFILT_READ, flags, 0, 0, (void *)ev);
192192

193193
if (kevent(kfd, &k, 1, NULL, 0, NULL) < 0) {
194-
zlog(ZLOG_ERROR, "kevent: unable to add event");
194+
zlog(ZLOG_ERROR, "kevent: unable to delete event");
195195
return -1;
196196
}
197197

198-
/* mark the vent as not registered */
198+
/* mark the event as not registered */
199199
ev->index = -1;
200200
return 0;
201201
}

0 commit comments

Comments
 (0)