Skip to content

Commit 562124f

Browse files
committed
Convert return value to bool for fpm_init()
The return value was always either 0 or -1
1 parent db8f54a commit 562124f

File tree

3 files changed

+7
-8
lines changed

3 files changed

+7
-8
lines changed

sapi/fpm/fpm/fpm.c

+4-4
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ struct fpm_globals_s fpm_globals = {
4141
.send_config_pipe = {0, 0},
4242
};
4343

44-
int fpm_init(int argc, char **argv, char *config, char *prefix, char *pid, int test_conf, int run_as_root, int force_daemon, int force_stderr) /* {{{ */
44+
bool fpm_init(int argc, char **argv, char *config, char *prefix, char *pid, int test_conf, int run_as_root, int force_daemon, int force_stderr) /* {{{ */
4545
{
4646
fpm_globals.argc = argc;
4747
fpm_globals.argv = argv;
@@ -70,19 +70,19 @@ int fpm_init(int argc, char **argv, char *config, char *prefix, char *pid, int t
7070
exit(FPM_EXIT_OK);
7171
} else {
7272
zlog(ZLOG_ERROR, "FPM initialization failed");
73-
return -1;
73+
return false;
7474
}
7575
}
7676

7777
if (!fpm_conf_write_pid()) {
7878
zlog(ZLOG_ERROR, "FPM initialization failed");
79-
return -1;
79+
return false;
8080
}
8181

8282
fpm_stdio_init_final();
8383
zlog(ZLOG_NOTICE, "fpm is running, pid %d", (int) fpm_globals.parent_pid);
8484

85-
return 0;
85+
return true;
8686
}
8787
/* }}} */
8888

sapi/fpm/fpm/fpm.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@
3535

3636

3737
int fpm_run(int *max_requests);
38-
int fpm_init(int argc, char **argv, char *config, char *prefix, char *pid, int test_conf, int run_as_root, int force_daemon, int force_stderr);
38+
bool fpm_init(int argc, char **argv, char *config, char *prefix, char *pid, int test_conf, int run_as_root, int force_daemon, int force_stderr);
3939

4040
struct fpm_globals_s {
4141
pid_t parent_pid;

sapi/fpm/fpm/fpm_main.c

+2-3
Original file line numberDiff line numberDiff line change
@@ -1530,7 +1530,6 @@ int main(int argc, char *argv[])
15301530
int force_stderr = 0;
15311531
int php_information = 0;
15321532
int php_allow_to_run_as_root = 0;
1533-
int ret;
15341533
#if ZEND_RC_DEBUG
15351534
bool old_rc_debug;
15361535
#endif
@@ -1777,13 +1776,13 @@ consult the installation file that came with this distribution, or visit \n\
17771776
zend_rc_debug = 0;
17781777
#endif
17791778

1780-
ret = fpm_init(argc, argv, fpm_config ? fpm_config : CGIG(fpm_config), fpm_prefix, fpm_pid, test_conf, php_allow_to_run_as_root, force_daemon, force_stderr);
1779+
bool fpm_status = fpm_init(argc, argv, fpm_config ? fpm_config : CGIG(fpm_config), fpm_prefix, fpm_pid, test_conf, php_allow_to_run_as_root, force_daemon, force_stderr);
17811780

17821781
#if ZEND_RC_DEBUG
17831782
zend_rc_debug = old_rc_debug;
17841783
#endif
17851784

1786-
if (ret < 0) {
1785+
if (!fpm_status) {
17871786

17881787
if (fpm_globals.send_config_pipe[1]) {
17891788
int writeval = 0;

0 commit comments

Comments
 (0)