Fix FPM kqueue handler fix remove callback typo

It avoids confusion on failed event.
php · Apr 23, 2022 · ff90d42 · ff90d42
1 parent 6186ecd
commit ff90d42
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/NEWS b/NEWS
@@ -14,6 +14,7 @@ PHP                                                                        NEWS
   . Fixed bug #76003 (FPM /status reports wrong number of active processe).
     (Jakub Zelenka)
   . Fixed bug #77023 (FPM cannot shutdown processes). (Jakub Zelenka)
+  . Fixed comment in kqueue remove callback log message. (David Carlier)
 
 - MySQLi:
   . Fixed bug GH-8267 (MySQLi uses unsupported format specifier on Windows).

diff --git a/sapi/fpm/fpm/events/kqueue.c b/sapi/fpm/fpm/events/kqueue.c
@@ -191,11 +191,11 @@ static int fpm_event_kqueue_remove(struct fpm_event_s *ev) /* {{{ */
 	EV_SET(&k, ev->fd, EVFILT_READ, flags, 0, 0, (void *)ev);
 
 	if (kevent(kfd, &k, 1, NULL, 0, NULL) < 0) {
-		zlog(ZLOG_ERROR, "kevent: unable to add event");
+		zlog(ZLOG_ERROR, "kevent: unable to delete event");
 		return -1;
 	}
 
-	/* mark the vent as not registered */
+	/* mark the event as not registered */
 	ev->index = -1;
 	return 0;
 }