ext/soap: SoapServer::getFunctions() does not need bailout handler wrapper

Girgias · Girgias · commit 81ccaacaa8de · 2024-06-16T23:05:30.000+01:00
As this methods never uses the bailout mechanism
diff --git a/ext/soap/soap.c b/ext/soap/soap.c
@@ -1102,9 +1102,7 @@ PHP_METHOD(SoapServer, getFunctions)
 		RETURN_THROWS();
 	}
 
-	SOAP_SERVER_BEGIN_CODE();
-
-	FETCH_THIS_SERVICE(service);
+	FETCH_THIS_SERVICE_NO_BAILOUT(service);
 
 	array_init(return_value);
 	if (service->type == SOAP_OBJECT) {
@@ -1129,8 +1127,6 @@ PHP_METHOD(SoapServer, getFunctions)
 			}
 		} ZEND_HASH_FOREACH_END();
 	}
-
-	SOAP_SERVER_END_CODE();
 }
 /* }}} */
 

Original file line number	Diff line number	Diff line change
`@@ -1102,9 +1102,7 @@ PHP_METHOD(SoapServer, getFunctions)`
`1102`	`1102`	`RETURN_THROWS();`
`1103`	`1103`	`}`
`1104`	`1104`
`1105`		`- SOAP_SERVER_BEGIN_CODE();`
`1106`		`-`
`1107`		`- FETCH_THIS_SERVICE(service);`
	`1105`	`+ FETCH_THIS_SERVICE_NO_BAILOUT(service);`
`1108`	`1106`
`1109`	`1107`	`array_init(return_value);`
`1110`	`1108`	`if (service->type == SOAP_OBJECT) {`
`@@ -1129,8 +1127,6 @@ PHP_METHOD(SoapServer, getFunctions)`
`1129`	`1127`	`}`
`1130`	`1128`	`} ZEND_HASH_FOREACH_END();`
`1131`	`1129`	`}`
`1132`		`-`
`1133`		`- SOAP_SERVER_END_CODE();`
`1134`	`1130`	`}`
`1135`	`1131`	`/* }}} */`
`1136`	`1132`