diff options
author | Dmitry Stogov <dmitry@php.net> | 2005-10-05 07:01:48 +0000 |
---|---|---|
committer | Dmitry Stogov <dmitry@php.net> | 2005-10-05 07:01:48 +0000 |
commit | 408337fc181265449f6be4edcf636ae26b61cc61 (patch) | |
tree | 2f5edab6c9886253bb43b972e4d344c88d5cde29 | |
parent | f20435d2b542ac41d95ae83728d42d106385b3db (diff) | |
download | php-git-408337fc181265449f6be4edcf636ae26b61cc61.tar.gz |
Fixed throwing exceptions derived from SoapFault
-rw-r--r-- | ext/soap/soap.c | 20 | ||||
-rwxr-xr-x | ext/soap/tests/server022.phpt | 40 |
2 files changed, 50 insertions, 10 deletions
diff --git a/ext/soap/soap.c b/ext/soap/soap.c index 27e1403ef9..74cb18050d 100644 --- a/ext/soap/soap.c +++ b/ext/soap/soap.c @@ -110,7 +110,7 @@ static void soap_error_handler(int error_num, const char *error_filename, const EG(current_execute_data) = _old_current_execute_data; \ if (EG(exception) == NULL || \ Z_TYPE_P(EG(exception)) != IS_OBJECT || \ - Z_OBJCE_P(EG(exception)) != soap_fault_class_entry) {\ + !instanceof_function(Z_OBJCE_P(EG(exception)), soap_fault_class_entry TSRMLS_CC)) {\ _bailout = 1;\ }\ } zend_end_try();\ @@ -1540,7 +1540,7 @@ PHP_METHOD(SoapServer, handle) } if (EG(exception)) { if (Z_TYPE_P(EG(exception)) == IS_OBJECT && - Z_OBJCE_P(EG(exception)) == soap_fault_class_entry) { + instanceof_function(Z_OBJCE_P(EG(exception)), soap_fault_class_entry TSRMLS_CC)) { soap_server_fault_ex(function, EG(exception), NULL TSRMLS_CC); } else { zval_dtor(&constructor); @@ -1573,7 +1573,7 @@ PHP_METHOD(SoapServer, handle) #ifdef ZEND_ENGINE_2 if (EG(exception)) { if (Z_TYPE_P(EG(exception)) == IS_OBJECT && - Z_OBJCE_P(EG(exception)) == soap_fault_class_entry) { + instanceof_function(Z_OBJCE_P(EG(exception)), soap_fault_class_entry TSRMLS_CC)) { soap_server_fault_ex(function, EG(exception), NULL TSRMLS_CC); } else { zval_dtor(&constructor); @@ -1643,7 +1643,7 @@ PHP_METHOD(SoapServer, handle) php_error_docref(NULL TSRMLS_CC, E_ERROR, "Function '%s' call failed", Z_STRVAL(h->function_name)); } if (Z_TYPE(h->retval) == IS_OBJECT && - Z_OBJCE(h->retval) == soap_fault_class_entry) { + instanceof_function(Z_OBJCE(h->retval), soap_fault_class_entry TSRMLS_CC)) { zval *headerfault = NULL, **tmp; if (zend_hash_find(Z_OBJPROP(h->retval), "headerfault", sizeof("headerfault"), (void**)&tmp) == SUCCESS && @@ -1654,7 +1654,7 @@ PHP_METHOD(SoapServer, handle) #ifdef ZEND_ENGINE_2 } else if (EG(exception)) { if (Z_TYPE_P(EG(exception)) == IS_OBJECT && - Z_OBJCE_P(EG(exception)) == soap_fault_class_entry) { + instanceof_function(Z_OBJCE_P(EG(exception)), soap_fault_class_entry TSRMLS_CC)) { zval *headerfault = NULL, **tmp; if (zend_hash_find(Z_OBJPROP_P(EG(exception)), "headerfault", sizeof("headerfault"), (void**)&tmp) == SUCCESS && @@ -1701,7 +1701,7 @@ PHP_METHOD(SoapServer, handle) #ifdef ZEND_ENGINE_2 if (EG(exception)) { if (Z_TYPE_P(EG(exception)) == IS_OBJECT && - Z_OBJCE_P(EG(exception)) == soap_fault_class_entry) { + instanceof_function(Z_OBJCE_P(EG(exception)), soap_fault_class_entry TSRMLS_CC)) { soap_server_fault_ex(function, EG(exception), NULL TSRMLS_CC); } else { if (soap_obj) {zval_ptr_dtor(&soap_obj);} @@ -1714,7 +1714,7 @@ PHP_METHOD(SoapServer, handle) char *response_name; if (Z_TYPE(retval) == IS_OBJECT && - Z_OBJCE(retval) == soap_fault_class_entry) { + instanceof_function(Z_OBJCE(retval), soap_fault_class_entry TSRMLS_CC)) { soap_server_fault_ex(function, &retval, NULL TSRMLS_CC); } @@ -2017,7 +2017,7 @@ PHP_FUNCTION(is_soap_fault) if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "z", &fault) == SUCCESS && Z_TYPE_P(fault) == IS_OBJECT && - Z_OBJCE_P(fault) == soap_fault_class_entry) { + instanceof_function(Z_OBJCE_P(fault), soap_fault_class_entry TSRMLS_CC)) { RETURN_TRUE; } RETURN_FALSE @@ -2439,7 +2439,7 @@ static void do_soap_call(zval* this_ptr, } #ifdef ZEND_ENGINE_2 if (Z_TYPE_P(return_value) == IS_OBJECT && - Z_OBJCE_P(return_value) == soap_fault_class_entry && + instanceof_function(Z_OBJCE_P(return_value), soap_fault_class_entry TSRMLS_CC) && (zend_hash_find(Z_OBJPROP_P(this_ptr), "_exceptions", sizeof("_exceptions"), (void **) &tmp) != SUCCESS || Z_TYPE_PP(tmp) != IS_BOOL || Z_LVAL_PP(tmp) != 0)) { zval *exception; @@ -3436,7 +3436,7 @@ static xmlDocPtr serialize_response_call(sdlFunctionPtr function, char *function xmlDocSetRootElement(doc, envelope); if (Z_TYPE_P(ret) == IS_OBJECT && - Z_OBJCE_P(ret) == soap_fault_class_entry) { + instanceof_function(Z_OBJCE_P(ret), soap_fault_class_entry TSRMLS_CC)) { char *detail_name; HashTable* prop; zval **tmp; diff --git a/ext/soap/tests/server022.phpt b/ext/soap/tests/server022.phpt new file mode 100755 index 0000000000..7b74d8923f --- /dev/null +++ b/ext/soap/tests/server022.phpt @@ -0,0 +1,40 @@ +--TEST-- +SOAP Server 22: user fault (through throw of subclass) +--SKIPIF-- +<?php require_once('skipif.inc'); ?> +--FILE-- +<?php +class MyFault extends SoapFault { + function __construct() { + parent::__construct("MyFault","My fault string"); + } +} + + +function test() { + throw new MyFault; +} + +$server = new soapserver(null,array('uri'=>"http://testuri.org")); +$server->addfunction("test"); + +$HTTP_RAW_POST_DATA = <<<EOF +<?xml version="1.0" encoding="ISO-8859-1"?> +<SOAP-ENV:Envelope + SOAP-ENV:encodingStyle="http://schemas.xmlsoap.org/soap/encoding/" + xmlns:SOAP-ENV="http://schemas.xmlsoap.org/soap/envelope/" + xmlns:xsd="http://www.w3.org/2001/XMLSchema" + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" + xmlns:si="http://soapinterop.org/xsd"> + <SOAP-ENV:Body> + <ns1:test xmlns:ns1="http://testuri.org"/> + </SOAP-ENV:Body> +</SOAP-ENV:Envelope> +EOF; + +$server->handle(); +echo "ok\n"; +?> +--EXPECT-- +<?xml version="1.0" encoding="UTF-8"?> +<SOAP-ENV:Envelope xmlns:SOAP-ENV="http://schemas.xmlsoap.org/soap/envelope/"><SOAP-ENV:Body><SOAP-ENV:Fault><faultcode>MyFault</faultcode><faultstring>My fault string</faultstring></SOAP-ENV:Fault></SOAP-ENV:Body></SOAP-ENV:Envelope> |