Skip to content

Commit a07d76c

Browse files
author
Jerome Loyet
committed
- Fixed bug #61295 (php-fpm should not fail with commented 'user' for non-root start)
1 parent 757e3fc commit a07d76c

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

NEWS

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,8 @@ PHP NEWS
5757
- FPM
5858
. Fixed bug #61045 (fpm don't send error log to fastcgi clients). (fat)
5959
. Fixed bug #61835 (php-fpm is not allowed to run as root). (fat)
60+
. Fixed bug #61295 (php-fpm should not fail with commented 'user'
61+
for non-root start). (fat)
6062

6163
- Libxml:
6264
. Fixed bug #61617 (Libxml tests failed(ht is already destroyed)).

sapi/fpm/fpm/fpm_conf.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -723,8 +723,8 @@ static int fpm_conf_process_all_pools() /* {{{ */
723723
}
724724
}
725725

726-
/* user */
727-
if (!wp->config->user) {
726+
/* alert if user is not set only if we are not root*/
727+
if (!wp->config->user && !geteuid()) {
728728
zlog(ZLOG_ALERT, "[pool %s] user has not been defined", wp->config->name);
729729
return -1;
730730
}

0 commit comments

Comments
 (0)