ext/soap: Don't call readfile() userland function

We can perform the operation directly, moreover there is no risk of a user disabling the readfile function and defining their own messing up what we are doing.

Author: Girgias

ext/soap/soap.c

@@ -1307,19 +1307,14 @@ PHP_METHOD(SoapServer, handle)
 			sapi_add_header(hdr, sizeof("Location: ")+strlen(service->sdl->source)-1, 1);
 			efree(hdr);
 */
-			zval readfile, readfile_ret, param;
 
 			sapi_add_header("Content-Type: text/xml; charset=utf-8", sizeof("Content-Type: text/xml; charset=utf-8")-1, 1);
-			ZVAL_STRING(&param, service->sdl->source);
-			ZVAL_STRING(&readfile, "readfile");
-			if (call_user_function(EG(function_table), NULL, &readfile, &readfile_ret, 1, &param ) == FAILURE) {
-				soap_server_fault("Server", "Couldn't find WSDL", NULL, NULL, NULL);
+			php_stream *stream = php_stream_open_wrapper_ex(service->sdl->source, "rb", REPORT_ERRORS, NULL, /* context */ NULL);
+			if (stream) {
+				php_stream_passthru(stream);
+				php_stream_close(stream);
 			}
 
-			zval_ptr_dtor(&param);
-			zval_ptr_dtor_str(&readfile);
-			zval_ptr_dtor(&readfile_ret);
-
 			SOAP_SERVER_END_CODE();
 			return;
 		} else {

ext/soap/tests/SoapServer/handle_non_existing_WSDL_from_get_query_disable_readfile.phpt

@@ -81,4 +81,3 @@ $wsdlFile = __DIR__ . '/test_handle_non_existent_wsdl.wsdl';
 @unlink($wsdlFile);
 ?>
 --EXPECT--
-Error: Invalid callback readfile, function "readfile" not found or invalid function name

ext/soap/tests/SoapServer/handle_non_existing_WSDL_from_get_query_redefine_readfile.phpt

@@ -85,4 +85,3 @@ $wsdlFile = __DIR__ . '/test_handle_non_existent_wsdl.wsdl';
 @unlink($wsdlFile);
 ?>
 --EXPECT--
-Exception: BOO