Fix ASAN reported leak in FPM config test

bukka · bukka · commit ba6987ae8bfb · 2023-01-12T11:55:19.000Z
diff --git a/sapi/fpm/fpm/fpm_main.c b/sapi/fpm/fpm/fpm_main.c
@@ -1798,7 +1798,8 @@ consult the installation file that came with this distribution, or visit \n\
 			zend_quiet_write(fpm_globals.send_config_pipe[1], &writeval, sizeof(writeval));
 			close(fpm_globals.send_config_pipe[1]);
 		}
-		return FPM_EXIT_CONFIG;
+		exit_status = FPM_EXIT_CONFIG;
+		goto out;
 	}
 
 	if (fpm_globals.send_config_pipe[1]) {
diff --git a/sapi/fpm/tests/bug68591-conf-test-group.phpt b/sapi/fpm/tests/bug68591-conf-test-group.phpt
@@ -14,6 +14,7 @@ $cfg = <<<EOT
 error_log = {{FILE:LOG}}
 [unconfined]
 listen = {{ADDR:UDS}}
+user = root
 group = aaaaaa
 pm = dynamic
 pm.max_children = 5
diff --git a/sapi/fpm/tests/bug68591-conf-test-listen-group.phpt b/sapi/fpm/tests/bug68591-conf-test-listen-group.phpt
@@ -15,6 +15,7 @@ error_log = {{FILE:LOG}}
 [unconfined]
 listen = {{ADDR:UDS}}
 listen.group = aaaaaa
+user = root
 pm = dynamic
 pm.max_children = 5
 pm.start_servers = 2
diff --git a/sapi/fpm/tests/bug68591-conf-test-listen-owner.phpt b/sapi/fpm/tests/bug68591-conf-test-listen-owner.phpt
@@ -15,6 +15,7 @@ error_log = {{FILE:LOG}}
 [unconfined]
 listen = {{ADDR:UDS}}
 listen.owner = aaaaaa
+user = root
 pm = dynamic
 pm.max_children = 5
 pm.start_servers = 2

Original file line number	Diff line number	Diff line change
`@@ -1798,7 +1798,8 @@ consult the installation file that came with this distribution, or visit \n\`
`1798`	`1798`	`zend_quiet_write(fpm_globals.send_config_pipe[1], &writeval, sizeof(writeval));`
`1799`	`1799`	`close(fpm_globals.send_config_pipe[1]);`
`1800`	`1800`	`}`
`1801`		`- return FPM_EXIT_CONFIG;`
	`1801`	`+ exit_status = FPM_EXIT_CONFIG;`
	`1802`	`+ goto out;`
`1802`	`1803`	`}`
`1803`	`1804`
`1804`	`1805`	`if (fpm_globals.send_config_pipe[1]) {`