diff options
Diffstat (limited to 'ext/xml')
31 files changed, 673 insertions, 692 deletions
diff --git a/ext/xml/compat.c b/ext/xml/compat.c index 56493db5a6..df0879dea4 100644 --- a/ext/xml/compat.c +++ b/ext/xml/compat.c @@ -1,6 +1,6 @@ /* +----------------------------------------------------------------------+ - | PHP Version 5 | + | PHP Version 7 | +----------------------------------------------------------------------+ | Copyright (c) 1997-2015 The PHP Group | +----------------------------------------------------------------------+ @@ -33,7 +33,7 @@ typedef struct _php_xml_ns { ((__ns) != NULL && strlen(__ns) == 5 && *(__ns) == 'x' && *((__ns)+1) == 'm' && \ *((__ns)+2) == 'l' && *((__ns)+3) == 'n' && *((__ns)+4) == 's') -static void +static void _qualify_namespace(XML_Parser parser, const xmlChar *name, const xmlChar *URI, xmlChar **qualified) { if (URI) { @@ -95,7 +95,7 @@ _start_element_handler_ns(void *user, const xmlChar *name, const xmlChar *prefix int i; int z = 0; int y = 0; - + if (nb_namespaces > 0 && parser->h_start_ns != NULL) { for (i = 0; i < nb_namespaces; i += 1) { parser->h_start_ns(parser->user, (const XML_Char *) namespaces[y], (const XML_Char *) namespaces[y+1]); @@ -103,7 +103,7 @@ _start_element_handler_ns(void *user, const xmlChar *name, const xmlChar *prefix } y = 0; } - + if (parser->h_start_element == NULL) { if (parser->h_default) { @@ -114,27 +114,27 @@ _start_element_handler_ns(void *user, const xmlChar *name, const xmlChar *prefix } else { qualified_name = xmlStrncatNew((xmlChar *)"<", name, xmlStrlen(name)); } - + if (namespaces) { int i, j; for (i = 0,j = 0;j < nb_namespaces;j++) { int ns_len; char *ns_string, *ns_prefix, *ns_url; - + ns_prefix = (char *) namespaces[i++]; ns_url = (char *) namespaces[i++]; - + if (ns_prefix) { ns_len = spprintf(&ns_string, 0, " xmlns:%s=\"%s\"", ns_prefix, ns_url); } else { ns_len = spprintf(&ns_string, 0, " xmlns=\"%s\"", ns_url); } qualified_name = xmlStrncat(qualified_name, (xmlChar *)ns_string, ns_len); - + efree(ns_string); } } - + if (attributes) { for (i = 0; i < nb_attributes; i += 1) { int att_len; @@ -155,7 +155,7 @@ _start_element_handler_ns(void *user, const xmlChar *name, const xmlChar *prefix qualified_name = xmlStrncat(qualified_name, (xmlChar *)att_string, att_len); qualified_name = xmlStrncat(qualified_name, (xmlChar *)att_value, att_valueend - att_value); qualified_name = xmlStrncat(qualified_name, (xmlChar *)"\"", 1); - + efree(att_string); } @@ -167,7 +167,7 @@ _start_element_handler_ns(void *user, const xmlChar *name, const xmlChar *prefix return; } _qualify_namespace(parser, name, URI, &qualified_name); - + if (attributes != NULL) { xmlChar *qualified_name_attr = NULL; attrs = safe_emalloc((nb_attributes * 2) + 1, sizeof(int *), 0); @@ -198,15 +198,6 @@ _start_element_handler_ns(void *user, const xmlChar *name, const xmlChar *prefix } static void -_namespace_handler(XML_Parser parser, xmlNsPtr nsptr) -{ - if (nsptr != NULL) { - _namespace_handler(parser, nsptr->next); - parser->h_end_ns(parser->user, nsptr->prefix); - } -} - -static void _end_element_handler(void *user, const xmlChar *name) { xmlChar *qualified_name; @@ -222,7 +213,7 @@ _end_element_handler(void *user, const xmlChar *name) } return; } - + qualified_name = xmlStrdup(name); parser->h_end_element(parser->user, (const XML_Char *) qualified_name); @@ -293,10 +284,10 @@ _pi_handler(void *user, const xmlChar *target, const xmlChar *data) } static void -_unparsed_entity_decl_handler(void *user, - const xmlChar *name, - const xmlChar *pub_id, - const xmlChar *sys_id, +_unparsed_entity_decl_handler(void *user, + const xmlChar *name, + const xmlChar *pub_id, + const xmlChar *sys_id, const xmlChar *notation) { XML_Parser parser = (XML_Parser) user; @@ -324,7 +315,7 @@ static void _build_comment(const xmlChar *data, int data_len, xmlChar **comment, int *comment_len) { *comment_len = data_len + 7; - + *comment = xmlMalloc(*comment_len + 1); memcpy(*comment, "<!--", 4); memcpy(*comment + 4, data, data_len); @@ -349,7 +340,7 @@ _comment_handler(void *user, const xmlChar *comment) } static void -_build_entity(const xmlChar *name, int len, xmlChar **entity, int *entity_len) +_build_entity(const xmlChar *name, int len, xmlChar **entity, int *entity_len) { *entity_len = len + 2; *entity = xmlMalloc(*entity_len + 1); @@ -368,7 +359,7 @@ _external_entity_ref_handler(void *user, const xmlChar *names, int type, const x return; } - parser->h_external_entity_ref(parser, names, "", sys_id, pub_id); + parser->h_external_entity_ref(parser, names, (XML_Char *) "", sys_id, pub_id); } static xmlEntityPtr @@ -388,7 +379,7 @@ _get_entity(void *user, const xmlChar *name) if (parser->h_default && ! (ret && ret->etype == XML_INTERNAL_PREDEFINED_ENTITY && parser->h_cdata)) { xmlChar *entity; int len; - + _build_entity(name, xmlStrlen(name), &entity, &len); parser->h_default(parser->user, (const xmlChar *) entity, len); xmlFree(entity); @@ -410,7 +401,7 @@ _get_entity(void *user, const xmlChar *name) return ret; } -static xmlSAXHandler +static xmlSAXHandler php_xml_compat_handlers = { NULL, /* internalSubset */ NULL, /* isStandalone */ @@ -443,16 +434,16 @@ php_xml_compat_handlers = { NULL, _start_element_handler_ns, _end_element_handler_ns, - NULL + NULL }; -PHPAPI XML_Parser +PHP_XML_API XML_Parser XML_ParserCreate(const XML_Char *encoding) { return XML_ParserCreate_MM(encoding, NULL, NULL); } -PHPAPI XML_Parser +PHP_XML_API XML_Parser XML_ParserCreateNS(const XML_Char *encoding, const XML_Char sep) { XML_Char tmp[2]; @@ -461,7 +452,7 @@ XML_ParserCreateNS(const XML_Char *encoding, const XML_Char sep) return XML_ParserCreate_MM(encoding, NULL, tmp); } -PHPAPI XML_Parser +PHP_XML_API XML_Parser XML_ParserCreate_MM(const XML_Char *encoding, const XML_Memory_Handling_Suite *memsuite, const XML_Char *sep) { XML_Parser parser; @@ -493,93 +484,93 @@ XML_ParserCreate_MM(const XML_Char *encoding, const XML_Memory_Handling_Suite *m parser->parser->sax2 = 1; parser->_ns_separator = xmlStrdup(sep); } else { - /* Reset flag as XML_SAX2_MAGIC is needed for xmlCreatePushParserCtxt + /* Reset flag as XML_SAX2_MAGIC is needed for xmlCreatePushParserCtxt so must be set in the handlers */ parser->parser->sax->initialized = 1; } return parser; } -PHPAPI void +PHP_XML_API void XML_SetUserData(XML_Parser parser, void *user) { parser->user = user; } -PHPAPI void * +PHP_XML_API void * XML_GetUserData(XML_Parser parser) { return parser->user; } -PHPAPI void +PHP_XML_API void XML_SetElementHandler(XML_Parser parser, XML_StartElementHandler start, XML_EndElementHandler end) { parser->h_start_element = start; parser->h_end_element = end; } -PHPAPI void +PHP_XML_API void XML_SetCharacterDataHandler(XML_Parser parser, XML_CharacterDataHandler cdata) { parser->h_cdata = cdata; } -PHPAPI void +PHP_XML_API void XML_SetProcessingInstructionHandler(XML_Parser parser, XML_ProcessingInstructionHandler pi) { parser->h_pi = pi; } -PHPAPI void +PHP_XML_API void XML_SetCommentHandler(XML_Parser parser, XML_CommentHandler comment) { parser->h_comment = comment; } -PHPAPI void +PHP_XML_API void XML_SetDefaultHandler(XML_Parser parser, XML_DefaultHandler d) { parser->h_default = d; } -PHPAPI void +PHP_XML_API void XML_SetUnparsedEntityDeclHandler(XML_Parser parser, XML_UnparsedEntityDeclHandler unparsed_decl) { parser->h_unparsed_entity_decl = unparsed_decl; } -PHPAPI void +PHP_XML_API void XML_SetNotationDeclHandler(XML_Parser parser, XML_NotationDeclHandler notation_decl) { parser->h_notation_decl = notation_decl; } -PHPAPI void +PHP_XML_API void XML_SetExternalEntityRefHandler(XML_Parser parser, XML_ExternalEntityRefHandler ext_entity) { parser->h_external_entity_ref = ext_entity; } -PHPAPI void +PHP_XML_API void XML_SetStartNamespaceDeclHandler(XML_Parser parser, XML_StartNamespaceDeclHandler start_ns) { parser->h_start_ns = start_ns; } -PHPAPI void +PHP_XML_API void XML_SetEndNamespaceDeclHandler(XML_Parser parser, XML_EndNamespaceDeclHandler end_ns) { parser->h_end_ns = end_ns; } -PHPAPI int +PHP_XML_API int XML_Parse(XML_Parser parser, const XML_Char *data, int data_len, int is_final) { int error; -/* The following is a hack to keep BC with PHP 4 while avoiding -the inifite loop in libxml <= 2.6.17 which occurs when no encoding +/* The following is a hack to keep BC with PHP 4 while avoiding +the inifite loop in libxml <= 2.6.17 which occurs when no encoding has been defined and none can be detected */ #if LIBXML_VERSION <= 20617 if (parser->parser->charset == XML_CHAR_ENCODING_NONE) { @@ -602,7 +593,7 @@ has been defined and none can be detected */ } #endif - error = xmlParseChunk(parser->parser, data, data_len, is_final); + error = xmlParseChunk(parser->parser, (char *) data, data_len, is_final); if (!error) { return 1; } else if (parser->parser->lastError.level > XML_ERR_WARNING ){ @@ -612,7 +603,7 @@ has been defined and none can be detected */ } } -PHPAPI int +PHP_XML_API int XML_GetErrorCode(XML_Parser parser) { return parser->parser->errNo; @@ -724,35 +715,35 @@ static const XML_Char *const error_mapping[] = { (const XML_Char *)"Missing encoding in text declaration" /* 101 */ }; -PHPAPI const XML_Char * +PHP_XML_API const XML_Char * XML_ErrorString(int code) { if (code < 0 || code >= (int)(sizeof(error_mapping) / sizeof(error_mapping[0]))) { - return "Unknown"; + return (const XML_Char *) "Unknown"; } return error_mapping[code]; } -PHPAPI int +PHP_XML_API int XML_GetCurrentLineNumber(XML_Parser parser) { return parser->parser->input->line; } -PHPAPI int +PHP_XML_API int XML_GetCurrentColumnNumber(XML_Parser parser) { return parser->parser->input->col; } -PHPAPI int +PHP_XML_API int XML_GetCurrentByteIndex(XML_Parser parser) { return parser->parser->input->consumed + (parser->parser->input->cur - parser->parser->input->base); } -PHPAPI int +PHP_XML_API int XML_GetCurrentByteCount(XML_Parser parser) { /* WARNING: this is identical to ByteIndex; it should probably @@ -761,12 +752,12 @@ XML_GetCurrentByteCount(XML_Parser parser) (parser->parser->input->cur - parser->parser->input->base); } -PHPAPI const XML_Char *XML_ExpatVersion(void) +PHP_XML_API const XML_Char *XML_ExpatVersion(void) { - return "1.0"; + return (const XML_Char *) "1.0"; } -PHPAPI void +PHP_XML_API void XML_ParserFree(XML_Parser parser) { if (parser->use_namespace) { diff --git a/ext/xml/config.m4 b/ext/xml/config.m4 index ebfc0471e0..812032bc6a 100644 --- a/ext/xml/config.m4 +++ b/ext/xml/config.m4 @@ -52,7 +52,7 @@ if test "$PHP_XML" != "no"; then AC_DEFINE(HAVE_LIBEXPAT, 1, [ ]) fi - PHP_NEW_EXTENSION(xml, xml.c $xml_extra_sources, $ext_shared) + PHP_NEW_EXTENSION(xml, xml.c $xml_extra_sources, $ext_shared,, -DZEND_ENABLE_STATIC_TSRMLS_CACHE=1) PHP_SUBST(XML_SHARED_LIBADD) PHP_INSTALL_HEADERS([ext/xml/]) AC_DEFINE(HAVE_XML, 1, [ ]) diff --git a/ext/xml/config.w32 b/ext/xml/config.w32 index 4ee0bd1602..54e5fc234b 100644 --- a/ext/xml/config.w32 +++ b/ext/xml/config.w32 @@ -4,9 +4,12 @@ ARG_WITH("xml", "XML support", "yes"); if (PHP_XML == "yes") { - if (PHP_LIBXML == "yes" - && ADD_EXTENSION_DEP('xml', 'libxml')) { - EXTENSION("xml", "xml.c compat.c"); + if (PHP_LIBXML == "yes" && + ADD_EXTENSION_DEP('xml', 'libxml') && + CHECK_HEADER_ADD_INCLUDE("libxml/parser.h", "CFLAGS_XML", PHP_PHP_BUILD + "\\include\\libxml2") && + CHECK_HEADER_ADD_INCLUDE("libxml/tree.h", "CFLAGS_XML", PHP_PHP_BUILD + "\\include\\libxml2") + ) { + EXTENSION("xml", "xml.c compat.c", null, "/DZEND_ENABLE_STATIC_TSRMLS_CACHE=1"); AC_DEFINE("HAVE_XML", 1, "XML support"); if (!PHP_XML_SHARED) { ADD_FLAG("CFLAGS_XML", "/D LIBXML_STATIC "); diff --git a/ext/xml/expat_compat.h b/ext/xml/expat_compat.h index e47bdcf8ef..420bedd19d 100644 --- a/ext/xml/expat_compat.h +++ b/ext/xml/expat_compat.h @@ -1,6 +1,6 @@ /* +----------------------------------------------------------------------+ - | PHP Version 5 | + | PHP Version 7 | +----------------------------------------------------------------------+ | Copyright (c) 1997-2015 The PHP Group | +----------------------------------------------------------------------+ @@ -27,6 +27,14 @@ #include <php_config.h> #endif +#ifdef PHP_WIN32 +# define PHP_XML_API __declspec(dllexport) +#elif defined(__GNUC__) && __GNUC__ >= 4 +# define PHP_XML_API __attribute__ ((visibility("default"))) +#else +# define PHP_XML_API +#endif + #if !defined(HAVE_LIBEXPAT) && defined(HAVE_LIBXML) #define LIBXML_EXPAT_COMPAT 1 @@ -121,29 +129,29 @@ enum XML_Content_Type { XML_CTYPE_SEQ }; -PHPAPI XML_Parser XML_ParserCreate(const XML_Char *); -PHPAPI XML_Parser XML_ParserCreateNS(const XML_Char *, const XML_Char); -PHPAPI XML_Parser XML_ParserCreate_MM(const XML_Char *, const XML_Memory_Handling_Suite *, const XML_Char *); -PHPAPI void XML_SetUserData(XML_Parser, void *); -PHPAPI void *XML_GetUserData(XML_Parser); -PHPAPI void XML_SetElementHandler(XML_Parser, XML_StartElementHandler, XML_EndElementHandler); -PHPAPI void XML_SetCharacterDataHandler(XML_Parser, XML_CharacterDataHandler); -PHPAPI void XML_SetProcessingInstructionHandler(XML_Parser, XML_ProcessingInstructionHandler); -PHPAPI void XML_SetDefaultHandler(XML_Parser, XML_DefaultHandler); -PHPAPI void XML_SetUnparsedEntityDeclHandler(XML_Parser, XML_UnparsedEntityDeclHandler); -PHPAPI void XML_SetNotationDeclHandler(XML_Parser, XML_NotationDeclHandler); -PHPAPI void XML_SetExternalEntityRefHandler(XML_Parser, XML_ExternalEntityRefHandler); -PHPAPI void XML_SetStartNamespaceDeclHandler(XML_Parser, XML_StartNamespaceDeclHandler); -PHPAPI void XML_SetEndNamespaceDeclHandler(XML_Parser, XML_EndNamespaceDeclHandler); -PHPAPI int XML_Parse(XML_Parser, const XML_Char *, int data_len, int is_final); -PHPAPI int XML_GetErrorCode(XML_Parser); -PHPAPI const XML_Char *XML_ErrorString(int); -PHPAPI int XML_GetCurrentLineNumber(XML_Parser); -PHPAPI int XML_GetCurrentColumnNumber(XML_Parser); -PHPAPI int XML_GetCurrentByteIndex(XML_Parser); -PHPAPI int XML_GetCurrentByteCount(XML_Parser); -PHPAPI const XML_Char *XML_ExpatVersion(void); -PHPAPI void XML_ParserFree(XML_Parser); +PHP_XML_API XML_Parser XML_ParserCreate(const XML_Char *); +PHP_XML_API XML_Parser XML_ParserCreateNS(const XML_Char *, const XML_Char); +PHP_XML_API XML_Parser XML_ParserCreate_MM(const XML_Char *, const XML_Memory_Handling_Suite *, const XML_Char *); +PHP_XML_API void XML_SetUserData(XML_Parser, void *); +PHP_XML_API void *XML_GetUserData(XML_Parser); +PHP_XML_API void XML_SetElementHandler(XML_Parser, XML_StartElementHandler, XML_EndElementHandler); +PHP_XML_API void XML_SetCharacterDataHandler(XML_Parser, XML_CharacterDataHandler); +PHP_XML_API void XML_SetProcessingInstructionHandler(XML_Parser, XML_ProcessingInstructionHandler); +PHP_XML_API void XML_SetDefaultHandler(XML_Parser, XML_DefaultHandler); +PHP_XML_API void XML_SetUnparsedEntityDeclHandler(XML_Parser, XML_UnparsedEntityDeclHandler); +PHP_XML_API void XML_SetNotationDeclHandler(XML_Parser, XML_NotationDeclHandler); +PHP_XML_API void XML_SetExternalEntityRefHandler(XML_Parser, XML_ExternalEntityRefHandler); +PHP_XML_API void XML_SetStartNamespaceDeclHandler(XML_Parser, XML_StartNamespaceDeclHandler); +PHP_XML_API void XML_SetEndNamespaceDeclHandler(XML_Parser, XML_EndNamespaceDeclHandler); +PHP_XML_API int XML_Parse(XML_Parser, const XML_Char *, int data_len, int is_final); +PHP_XML_API int XML_GetErrorCode(XML_Parser); +PHP_XML_API const XML_Char *XML_ErrorString(int); +PHP_XML_API int XML_GetCurrentLineNumber(XML_Parser); +PHP_XML_API int XML_GetCurrentColumnNumber(XML_Parser); +PHP_XML_API int XML_GetCurrentByteIndex(XML_Parser); +PHP_XML_API int XML_GetCurrentByteCount(XML_Parser); +PHP_XML_API const XML_Char *XML_ExpatVersion(void); +PHP_XML_API void XML_ParserFree(XML_Parser); #elif defined(HAVE_LIBEXPAT) #include <expat.h> diff --git a/ext/xml/php_xml.h b/ext/xml/php_xml.h index fdce3a4cf0..8f92eefe1d 100644 --- a/ext/xml/php_xml.h +++ b/ext/xml/php_xml.h @@ -1,6 +1,6 @@ /* +----------------------------------------------------------------------+ - | PHP Version 5 | + | PHP Version 7 | +----------------------------------------------------------------------+ | Copyright (c) 1997-2015 The PHP Group | +----------------------------------------------------------------------+ @@ -26,6 +26,10 @@ #ifdef HAVE_XML extern zend_module_entry xml_module_entry; #define xml_module_ptr &xml_module_entry + +#include "php_version.h" +#define PHP_XML_VERSION PHP_VERSION + #else #define xml_module_ptr NULL #endif @@ -34,7 +38,6 @@ extern zend_module_entry xml_module_entry; #include "ext/xml/expat_compat.h" - #ifdef XML_UNICODE #error "UTF-16 Unicode support not implemented!" #endif @@ -44,22 +47,22 @@ ZEND_BEGIN_MODULE_GLOBALS(xml) ZEND_END_MODULE_GLOBALS(xml) typedef struct { - int index; int case_folding; XML_Parser parser; XML_Char *target_encoding; - zval *startElementHandler; - zval *endElementHandler; - zval *characterDataHandler; - zval *processingInstructionHandler; - zval *defaultHandler; - zval *unparsedEntityDeclHandler; - zval *notationDeclHandler; - zval *externalEntityRefHandler; - zval *unknownEncodingHandler; - zval *startNamespaceDeclHandler; - zval *endNamespaceDeclHandler; + zval index; + zval startElementHandler; + zval endElementHandler; + zval characterDataHandler; + zval processingInstructionHandler; + zval defaultHandler; + zval unparsedEntityDeclHandler; + zval notationDeclHandler; + zval externalEntityRefHandler; + zval unknownEncodingHandler; + zval startNamespaceDeclHandler; + zval endNamespaceDeclHandler; zend_function *startElementPtr; zend_function *endElementPtr; @@ -73,14 +76,14 @@ typedef struct { zend_function *startNamespaceDeclPtr; zend_function *endNamespaceDeclPtr; - zval *object; + zval object; - zval *data; - zval *info; + zval data; + zval info; int level; int toffset; int curtag; - zval **ctag; + zval *ctag; char **ltags; int lastwasopen; int skipwhite; @@ -133,18 +136,18 @@ PHP_FUNCTION(utf8_encode); PHP_FUNCTION(utf8_decode); PHP_FUNCTION(xml_parse_into_struct); -PHPAPI char *_xml_zval_strdup(zval *val); -PHPAPI char *xml_utf8_decode(const XML_Char *, int, int *, const XML_Char *); -PHPAPI char *xml_utf8_encode(const char *s, int len, int *newlen, const XML_Char *encoding); +PHP_XML_API char *_xml_zval_strdup(zval *); +PHP_XML_API zend_string *xml_utf8_decode(const XML_Char *, size_t, const XML_Char *); +PHP_XML_API zend_string *xml_utf8_encode(const char *, size_t, const XML_Char *); #endif /* HAVE_LIBEXPAT */ #define phpext_xml_ptr xml_module_ptr -#ifdef ZTS -#define XML(v) TSRMG(xml_globals_id, zend_xml_globals *, v) -#else -#define XML(v) (xml_globals.v) +#define XML(v) ZEND_MODULE_GLOBALS_ACCESSOR(xml, v) + +#if defined(ZTS) && defined(COMPILE_DL_XML) +ZEND_TSRMLS_CACHE_EXTERN(); #endif #endif /* PHP_XML_H */ diff --git a/ext/xml/tests/bug32001.phpt b/ext/xml/tests/bug32001.phpt index ed93582081..2e9e67d4ca 100644 --- a/ext/xml/tests/bug32001.phpt +++ b/ext/xml/tests/bug32001.phpt @@ -16,7 +16,7 @@ class testcase { private $tags; private $chunk_size; - function testcase($enc, $chunk_size = 0, $bom = 0, $omit_prologue = 0) { + function __construct($enc, $chunk_size = 0, $bom = 0, $omit_prologue = 0) { $this->encoding = $enc; $this->chunk_size = $chunk_size; $this->bom = $bom; diff --git a/ext/xml/tests/xml_error_string_variation1.phpt b/ext/xml/tests/xml_error_string_variation1.phpt index 40b542c141..c97d995cf5 100644 --- a/ext/xml/tests/xml_error_string_variation1.phpt +++ b/ext/xml/tests/xml_error_string_variation1.phpt @@ -1,7 +1,7 @@ --TEST-- Test xml_error_string() function : usage variations - test different types for code --SKIPIF-- -<?php +<?php if (PHP_INT_SIZE != 8) die("skip this test is for 64-bit only"); if (!extension_loaded("xml")) { print "skip - XML extension not loaded"; } @@ -103,27 +103,27 @@ string(8) "No error" Arg value Array -Warning: xml_error_string() expects parameter 1 to be long, array given in %s on line %d +Warning: xml_error_string() expects parameter 1 to be integer, array given in %s on line %d NULL Arg value Array -Warning: xml_error_string() expects parameter 1 to be long, array given in %s on line %d +Warning: xml_error_string() expects parameter 1 to be integer, array given in %s on line %d NULL Arg value Array -Warning: xml_error_string() expects parameter 1 to be long, array given in %s on line %d +Warning: xml_error_string() expects parameter 1 to be integer, array given in %s on line %d NULL Arg value Array -Warning: xml_error_string() expects parameter 1 to be long, array given in %s on line %d +Warning: xml_error_string() expects parameter 1 to be integer, array given in %s on line %d NULL Arg value Array -Warning: xml_error_string() expects parameter 1 to be long, array given in %s on line %d +Warning: xml_error_string() expects parameter 1 to be integer, array given in %s on line %d NULL Arg value @@ -146,27 +146,27 @@ string(8) "No error" Arg value -Warning: xml_error_string() expects parameter 1 to be long, string given in %s on line %d +Warning: xml_error_string() expects parameter 1 to be integer, string given in %s on line %d NULL Arg value -Warning: xml_error_string() expects parameter 1 to be long, string given in %s on line %d +Warning: xml_error_string() expects parameter 1 to be integer, string given in %s on line %d NULL Arg value string -Warning: xml_error_string() expects parameter 1 to be long, string given in %s on line %d +Warning: xml_error_string() expects parameter 1 to be integer, string given in %s on line %d NULL Arg value string -Warning: xml_error_string() expects parameter 1 to be long, string given in %s on line %d +Warning: xml_error_string() expects parameter 1 to be integer, string given in %s on line %d NULL Arg value Some Ascii Data -Warning: xml_error_string() expects parameter 1 to be long, object given in %s on line %d +Warning: xml_error_string() expects parameter 1 to be integer, object given in %s on line %d NULL Arg value diff --git a/ext/xml/tests/xml_get_current_byte_index_variation1.phpt b/ext/xml/tests/xml_get_current_byte_index_variation1.phpt index b5c83d78e9..2214c86ffa 100644 --- a/ext/xml/tests/xml_get_current_byte_index_variation1.phpt +++ b/ext/xml/tests/xml_get_current_byte_index_variation1.phpt @@ -119,27 +119,27 @@ NULL Arg value 10.5 -Warning: xml_get_current_byte_index() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_get_current_byte_index() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value -10.5 -Warning: xml_get_current_byte_index() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_get_current_byte_index() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 101234567000 -Warning: xml_get_current_byte_index() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_get_current_byte_index() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 1.07654321E-9 -Warning: xml_get_current_byte_index() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_get_current_byte_index() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 0.5 -Warning: xml_get_current_byte_index() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_get_current_byte_index() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value Array diff --git a/ext/xml/tests/xml_get_current_column_number_variation1.phpt b/ext/xml/tests/xml_get_current_column_number_variation1.phpt index 0eeb8d078e..afcac48c4b 100644 --- a/ext/xml/tests/xml_get_current_column_number_variation1.phpt +++ b/ext/xml/tests/xml_get_current_column_number_variation1.phpt @@ -120,27 +120,27 @@ NULL Arg value 10.5 -Warning: xml_get_current_column_number() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_get_current_column_number() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value -10.5 -Warning: xml_get_current_column_number() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_get_current_column_number() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 101234567000 -Warning: xml_get_current_column_number() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_get_current_column_number() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 1.07654321E-9 -Warning: xml_get_current_column_number() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_get_current_column_number() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 0.5 -Warning: xml_get_current_column_number() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_get_current_column_number() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value Array diff --git a/ext/xml/tests/xml_get_current_line_number_variation1.phpt b/ext/xml/tests/xml_get_current_line_number_variation1.phpt index af41c6f9e8..1b92c2784d 100644 --- a/ext/xml/tests/xml_get_current_line_number_variation1.phpt +++ b/ext/xml/tests/xml_get_current_line_number_variation1.phpt @@ -119,27 +119,27 @@ NULL Arg value 10.5 -Warning: xml_get_current_line_number() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_get_current_line_number() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value -10.5 -Warning: xml_get_current_line_number() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_get_current_line_number() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 101234567000 -Warning: xml_get_current_line_number() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_get_current_line_number() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 1.07654321E-9 -Warning: xml_get_current_line_number() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_get_current_line_number() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 0.5 -Warning: xml_get_current_line_number() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_get_current_line_number() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value Array diff --git a/ext/xml/tests/xml_get_error_code_variation1.phpt b/ext/xml/tests/xml_get_error_code_variation1.phpt index 474921406a..62e9ac9fb3 100644 --- a/ext/xml/tests/xml_get_error_code_variation1.phpt +++ b/ext/xml/tests/xml_get_error_code_variation1.phpt @@ -119,27 +119,27 @@ NULL Arg value 10.5 -Warning: xml_get_error_code() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_get_error_code() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value -10.5 -Warning: xml_get_error_code() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_get_error_code() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 101234567000 -Warning: xml_get_error_code() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_get_error_code() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 1.07654321E-9 -Warning: xml_get_error_code() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_get_error_code() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 0.5 -Warning: xml_get_error_code() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_get_error_code() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value Array diff --git a/ext/xml/tests/xml_parse_into_struct_variation1.phpt b/ext/xml/tests/xml_parse_into_struct_variation1.phpt index fa1e8d2990..39ce62a5da 100644 --- a/ext/xml/tests/xml_parse_into_struct_variation1.phpt +++ b/ext/xml/tests/xml_parse_into_struct_variation1.phpt @@ -120,27 +120,27 @@ NULL Arg value 10.5 -Warning: xml_parse_into_struct() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_parse_into_struct() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value -10.5 -Warning: xml_parse_into_struct() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_parse_into_struct() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 101234567000 -Warning: xml_parse_into_struct() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_parse_into_struct() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 1.07654321E-9 -Warning: xml_parse_into_struct() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_parse_into_struct() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 0.5 -Warning: xml_parse_into_struct() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_parse_into_struct() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value Array diff --git a/ext/xml/tests/xml_parse_variation1.phpt b/ext/xml/tests/xml_parse_variation1.phpt index cf36666499..34afcd87b5 100644 --- a/ext/xml/tests/xml_parse_variation1.phpt +++ b/ext/xml/tests/xml_parse_variation1.phpt @@ -121,27 +121,27 @@ NULL Arg value 10.5 -Warning: xml_parse() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_parse() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value -10.5 -Warning: xml_parse() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_parse() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 101234567000 -Warning: xml_parse() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_parse() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 1.07654321E-9 -Warning: xml_parse() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_parse() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 0.5 -Warning: xml_parse() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_parse() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value Array diff --git a/ext/xml/tests/xml_parser_free_variation1.phpt b/ext/xml/tests/xml_parser_free_variation1.phpt index 0ecb1092c8..7c6468a827 100644 --- a/ext/xml/tests/xml_parser_free_variation1.phpt +++ b/ext/xml/tests/xml_parser_free_variation1.phpt @@ -119,27 +119,27 @@ NULL Arg value 10.5 -Warning: xml_parser_free() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_parser_free() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value -10.5 -Warning: xml_parser_free() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_parser_free() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 101234567000 -Warning: xml_parser_free() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_parser_free() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 1.07654321E-9 -Warning: xml_parser_free() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_parser_free() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 0.5 -Warning: xml_parser_free() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_parser_free() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value Array diff --git a/ext/xml/tests/xml_parser_get_option_variation1.phpt b/ext/xml/tests/xml_parser_get_option_variation1.phpt index b53f803188..002bff6cd8 100644 --- a/ext/xml/tests/xml_parser_get_option_variation1.phpt +++ b/ext/xml/tests/xml_parser_get_option_variation1.phpt @@ -120,27 +120,27 @@ NULL Arg value 10.5 -Warning: xml_parser_get_option() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_parser_get_option() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value -10.5 -Warning: xml_parser_get_option() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_parser_get_option() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 101234567000 -Warning: xml_parser_get_option() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_parser_get_option() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 1.07654321E-9 -Warning: xml_parser_get_option() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_parser_get_option() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 0.5 -Warning: xml_parser_get_option() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_parser_get_option() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value Array diff --git a/ext/xml/tests/xml_parser_get_option_variation2.phpt b/ext/xml/tests/xml_parser_get_option_variation2.phpt index 2341c413f1..e741b9fa65 100644 --- a/ext/xml/tests/xml_parser_get_option_variation2.phpt +++ b/ext/xml/tests/xml_parser_get_option_variation2.phpt @@ -1,7 +1,7 @@ --TEST-- Test xml_parser_get_option() function : usage variations --SKIPIF-- -<?php +<?php if (PHP_INT_SIZE != 8) die("skip this test is for 64-bit only"); if (!extension_loaded("xml")) { print "skip - XML extension not loaded"; } @@ -134,27 +134,27 @@ bool(false) Arg value Array -Warning: xml_parser_get_option() expects parameter 2 to be long, array given in %s on line %d +Warning: xml_parser_get_option() expects parameter 2 to be integer, array given in %s on line %d NULL Arg value Array -Warning: xml_parser_get_option() expects parameter 2 to be long, array given in %s on line %d +Warning: xml_parser_get_option() expects parameter 2 to be integer, array given in %s on line %d NULL Arg value Array -Warning: xml_parser_get_option() expects parameter 2 to be long, array given in %s on line %d +Warning: xml_parser_get_option() expects parameter 2 to be integer, array given in %s on line %d NULL Arg value Array -Warning: xml_parser_get_option() expects parameter 2 to be long, array given in %s on line %d +Warning: xml_parser_get_option() expects parameter 2 to be integer, array given in %s on line %d NULL Arg value Array -Warning: xml_parser_get_option() expects parameter 2 to be long, array given in %s on line %d +Warning: xml_parser_get_option() expects parameter 2 to be integer, array given in %s on line %d NULL Arg value @@ -185,32 +185,32 @@ bool(false) Arg value -Warning: xml_parser_get_option() expects parameter 2 to be long, string given in %s on line %d +Warning: xml_parser_get_option() expects parameter 2 to be integer, string given in %s on line %d NULL Arg value -Warning: xml_parser_get_option() expects parameter 2 to be long, string given in %s on line %d +Warning: xml_parser_get_option() expects parameter 2 to be integer, string given in %s on line %d NULL Arg value string -Warning: xml_parser_get_option() expects parameter 2 to be long, string given in %s on line %d +Warning: xml_parser_get_option() expects parameter 2 to be integer, string given in %s on line %d NULL Arg value string -Warning: xml_parser_get_option() expects parameter 2 to be long, string given in %s on line %d +Warning: xml_parser_get_option() expects parameter 2 to be integer, string given in %s on line %d NULL Arg value Some Ascii Data -Warning: xml_parser_get_option() expects parameter 2 to be long, object given in %s on line %d +Warning: xml_parser_get_option() expects parameter 2 to be integer, object given in %s on line %d NULL Arg value Resource id %s -Warning: xml_parser_get_option() expects parameter 2 to be long, resource given in %s on line %d +Warning: xml_parser_get_option() expects parameter 2 to be integer, resource given in %s on line %d NULL Arg value diff --git a/ext/xml/tests/xml_parser_set_option_variation1.phpt b/ext/xml/tests/xml_parser_set_option_variation1.phpt index 6cfce0b4e3..14452dbe21 100644 --- a/ext/xml/tests/xml_parser_set_option_variation1.phpt +++ b/ext/xml/tests/xml_parser_set_option_variation1.phpt @@ -120,27 +120,27 @@ NULL Arg value 10.5 -Warning: xml_parser_set_option() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_parser_set_option() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value -10.5 -Warning: xml_parser_set_option() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_parser_set_option() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 101234567000 -Warning: xml_parser_set_option() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_parser_set_option() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 1.07654321E-9 -Warning: xml_parser_set_option() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_parser_set_option() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 0.5 -Warning: xml_parser_set_option() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_parser_set_option() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value Array diff --git a/ext/xml/tests/xml_parser_set_option_variation2.phpt b/ext/xml/tests/xml_parser_set_option_variation2.phpt index 9900d6fe6a..a367ede171 100644 --- a/ext/xml/tests/xml_parser_set_option_variation2.phpt +++ b/ext/xml/tests/xml_parser_set_option_variation2.phpt @@ -2,6 +2,7 @@ Test xml_parser_set_option() function : usage variations --SKIPIF-- <?php +if (PHP_INT_SIZE != 8) die("skip this test is for 64-bit only"); if (!extension_loaded("xml")) { print "skip - XML extension not loaded"; } @@ -128,27 +129,27 @@ bool(false) Arg value Array -Warning: xml_parser_set_option() expects parameter 2 to be long, array given in %s on line %d +Warning: xml_parser_set_option() expects parameter 2 to be integer, array given in %s on line %d NULL Arg value Array -Warning: xml_parser_set_option() expects parameter 2 to be long, array given in %s on line %d +Warning: xml_parser_set_option() expects parameter 2 to be integer, array given in %s on line %d NULL Arg value Array -Warning: xml_parser_set_option() expects parameter 2 to be long, array given in %s on line %d +Warning: xml_parser_set_option() expects parameter 2 to be integer, array given in %s on line %d NULL Arg value Array -Warning: xml_parser_set_option() expects parameter 2 to be long, array given in %s on line %d +Warning: xml_parser_set_option() expects parameter 2 to be integer, array given in %s on line %d NULL Arg value Array -Warning: xml_parser_set_option() expects parameter 2 to be long, array given in %s on line %d +Warning: xml_parser_set_option() expects parameter 2 to be integer, array given in %s on line %d NULL Arg value @@ -179,27 +180,27 @@ bool(false) Arg value -Warning: xml_parser_set_option() expects parameter 2 to be long, string given in %s on line %d +Warning: xml_parser_set_option() expects parameter 2 to be integer, string given in %s on line %d NULL Arg value -Warning: xml_parser_set_option() expects parameter 2 to be long, string given in %s on line %d +Warning: xml_parser_set_option() expects parameter 2 to be integer, string given in %s on line %d NULL Arg value string -Warning: xml_parser_set_option() expects parameter 2 to be long, string given in %s on line %d +Warning: xml_parser_set_option() expects parameter 2 to be integer, string given in %s on line %d NULL Arg value string -Warning: xml_parser_set_option() expects parameter 2 to be long, string given in %s on line %d +Warning: xml_parser_set_option() expects parameter 2 to be integer, string given in %s on line %d NULL Arg value Some Ascii Data -Warning: xml_parser_set_option() expects parameter 2 to be long, object given in %s on line %d +Warning: xml_parser_set_option() expects parameter 2 to be integer, object given in %s on line %d NULL Arg value diff --git a/ext/xml/tests/xml_set_character_data_handler_variation1.phpt b/ext/xml/tests/xml_set_character_data_handler_variation1.phpt index 40cff9193f..3e9f79768a 100644 --- a/ext/xml/tests/xml_set_character_data_handler_variation1.phpt +++ b/ext/xml/tests/xml_set_character_data_handler_variation1.phpt @@ -124,27 +124,27 @@ NULL Arg value 10.5 -Warning: xml_set_character_data_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_character_data_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value -10.5 -Warning: xml_set_character_data_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_character_data_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 101234567000 -Warning: xml_set_character_data_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_character_data_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 1.07654321E-9 -Warning: xml_set_character_data_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_character_data_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 0.5 -Warning: xml_set_character_data_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_character_data_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value Array diff --git a/ext/xml/tests/xml_set_default_handler_variation1.phpt b/ext/xml/tests/xml_set_default_handler_variation1.phpt index 35089a066a..afd0b6ed98 100644 --- a/ext/xml/tests/xml_set_default_handler_variation1.phpt +++ b/ext/xml/tests/xml_set_default_handler_variation1.phpt @@ -124,27 +124,27 @@ NULL Arg value 10.5 -Warning: xml_set_default_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_default_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value -10.5 -Warning: xml_set_default_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_default_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 101234567000 -Warning: xml_set_default_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_default_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 1.07654321E-9 -Warning: xml_set_default_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_default_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 0.5 -Warning: xml_set_default_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_default_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value Array diff --git a/ext/xml/tests/xml_set_element_handler_variation1.phpt b/ext/xml/tests/xml_set_element_handler_variation1.phpt index 225cf770b3..336237ab84 100644 --- a/ext/xml/tests/xml_set_element_handler_variation1.phpt +++ b/ext/xml/tests/xml_set_element_handler_variation1.phpt @@ -124,27 +124,27 @@ NULL Arg value 10.5 -Warning: xml_set_element_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_element_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value -10.5 -Warning: xml_set_element_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_element_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 101234567000 -Warning: xml_set_element_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_element_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 1.07654321E-9 -Warning: xml_set_element_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_element_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 0.5 -Warning: xml_set_element_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_element_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value Array diff --git a/ext/xml/tests/xml_set_end_namespace_decl_handler_variation1.phpt b/ext/xml/tests/xml_set_end_namespace_decl_handler_variation1.phpt index c7c2f6f176..eb90d99404 100644 --- a/ext/xml/tests/xml_set_end_namespace_decl_handler_variation1.phpt +++ b/ext/xml/tests/xml_set_end_namespace_decl_handler_variation1.phpt @@ -124,27 +124,27 @@ NULL Arg value 10.5 -Warning: xml_set_end_namespace_decl_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_end_namespace_decl_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value -10.5 -Warning: xml_set_end_namespace_decl_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_end_namespace_decl_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 101234567000 -Warning: xml_set_end_namespace_decl_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_end_namespace_decl_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 1.07654321E-9 -Warning: xml_set_end_namespace_decl_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_end_namespace_decl_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 0.5 -Warning: xml_set_end_namespace_decl_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_end_namespace_decl_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value Array diff --git a/ext/xml/tests/xml_set_external_entity_ref_handler_variation1.phpt b/ext/xml/tests/xml_set_external_entity_ref_handler_variation1.phpt index 10835b33b2..890374ea24 100644 --- a/ext/xml/tests/xml_set_external_entity_ref_handler_variation1.phpt +++ b/ext/xml/tests/xml_set_external_entity_ref_handler_variation1.phpt @@ -124,27 +124,27 @@ NULL Arg value 10.5 -Warning: xml_set_external_entity_ref_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_external_entity_ref_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value -10.5 -Warning: xml_set_external_entity_ref_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_external_entity_ref_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 101234567000 -Warning: xml_set_external_entity_ref_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_external_entity_ref_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 1.07654321E-9 -Warning: xml_set_external_entity_ref_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_external_entity_ref_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 0.5 -Warning: xml_set_external_entity_ref_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_external_entity_ref_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value Array diff --git a/ext/xml/tests/xml_set_notation_decl_handler_variation1.phpt b/ext/xml/tests/xml_set_notation_decl_handler_variation1.phpt index d41e691edc..97172658f9 100644 --- a/ext/xml/tests/xml_set_notation_decl_handler_variation1.phpt +++ b/ext/xml/tests/xml_set_notation_decl_handler_variation1.phpt @@ -124,27 +124,27 @@ NULL Arg value 10.5 -Warning: xml_set_notation_decl_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_notation_decl_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value -10.5 -Warning: xml_set_notation_decl_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_notation_decl_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 101234567000 -Warning: xml_set_notation_decl_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_notation_decl_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 1.07654321E-9 -Warning: xml_set_notation_decl_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_notation_decl_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 0.5 -Warning: xml_set_notation_decl_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_notation_decl_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value Array diff --git a/ext/xml/tests/xml_set_object_variation1.phpt b/ext/xml/tests/xml_set_object_variation1.phpt index 851dfaff10..fe06f9a248 100644 --- a/ext/xml/tests/xml_set_object_variation1.phpt +++ b/ext/xml/tests/xml_set_object_variation1.phpt @@ -120,27 +120,27 @@ NULL Arg value 10.5 -Warning: xml_set_object() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_object() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value -10.5 -Warning: xml_set_object() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_object() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 101234567000 -Warning: xml_set_object() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_object() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 1.07654321E-9 -Warning: xml_set_object() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_object() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 0.5 -Warning: xml_set_object() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_object() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value Array diff --git a/ext/xml/tests/xml_set_object_variation2.phpt b/ext/xml/tests/xml_set_object_variation2.phpt index 0844661786..bca3472a07 100644 --- a/ext/xml/tests/xml_set_object_variation2.phpt +++ b/ext/xml/tests/xml_set_object_variation2.phpt @@ -114,27 +114,27 @@ NULL Arg value 10.5 -Warning: xml_set_object() expects parameter 2 to be object, double given in %s on line %d +Warning: xml_set_object() expects parameter 2 to be object, float given in %s on line %d NULL Arg value -10.5 -Warning: xml_set_object() expects parameter 2 to be object, double given in %s on line %d +Warning: xml_set_object() expects parameter 2 to be object, float given in %s on line %d NULL Arg value 101234567000 -Warning: xml_set_object() expects parameter 2 to be object, double given in %s on line %d +Warning: xml_set_object() expects parameter 2 to be object, float given in %s on line %d NULL Arg value 1.07654321E-9 -Warning: xml_set_object() expects parameter 2 to be object, double given in %s on line %d +Warning: xml_set_object() expects parameter 2 to be object, float given in %s on line %d NULL Arg value 0.5 -Warning: xml_set_object() expects parameter 2 to be object, double given in %s on line %d +Warning: xml_set_object() expects parameter 2 to be object, float given in %s on line %d NULL Arg value Array diff --git a/ext/xml/tests/xml_set_processing_instruction_handler_variation1.phpt b/ext/xml/tests/xml_set_processing_instruction_handler_variation1.phpt index 9da14cc5fd..957458275b 100644 --- a/ext/xml/tests/xml_set_processing_instruction_handler_variation1.phpt +++ b/ext/xml/tests/xml_set_processing_instruction_handler_variation1.phpt @@ -124,27 +124,27 @@ NULL Arg value 10.5 -Warning: xml_set_processing_instruction_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_processing_instruction_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value -10.5 -Warning: xml_set_processing_instruction_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_processing_instruction_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 101234567000 -Warning: xml_set_processing_instruction_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_processing_instruction_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 1.07654321E-9 -Warning: xml_set_processing_instruction_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_processing_instruction_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 0.5 -Warning: xml_set_processing_instruction_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_processing_instruction_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value Array diff --git a/ext/xml/tests/xml_set_start_namespace_decl_handler_variation1.phpt b/ext/xml/tests/xml_set_start_namespace_decl_handler_variation1.phpt index 684a0fdac4..0a80b21a76 100644 --- a/ext/xml/tests/xml_set_start_namespace_decl_handler_variation1.phpt +++ b/ext/xml/tests/xml_set_start_namespace_decl_handler_variation1.phpt @@ -124,27 +124,27 @@ NULL Arg value 10.5 -Warning: xml_set_start_namespace_decl_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_start_namespace_decl_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value -10.5 -Warning: xml_set_start_namespace_decl_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_start_namespace_decl_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 101234567000 -Warning: xml_set_start_namespace_decl_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_start_namespace_decl_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 1.07654321E-9 -Warning: xml_set_start_namespace_decl_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_start_namespace_decl_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 0.5 -Warning: xml_set_start_namespace_decl_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_start_namespace_decl_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value Array diff --git a/ext/xml/tests/xml_set_unparsed_entity_decl_handler_variation1.phpt b/ext/xml/tests/xml_set_unparsed_entity_decl_handler_variation1.phpt index 1804115021..ec1061e59b 100644 --- a/ext/xml/tests/xml_set_unparsed_entity_decl_handler_variation1.phpt +++ b/ext/xml/tests/xml_set_unparsed_entity_decl_handler_variation1.phpt @@ -124,27 +124,27 @@ NULL Arg value 10.5 -Warning: xml_set_unparsed_entity_decl_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_unparsed_entity_decl_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value -10.5 -Warning: xml_set_unparsed_entity_decl_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_unparsed_entity_decl_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 101234567000 -Warning: xml_set_unparsed_entity_decl_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_unparsed_entity_decl_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 1.07654321E-9 -Warning: xml_set_unparsed_entity_decl_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_unparsed_entity_decl_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value 0.5 -Warning: xml_set_unparsed_entity_decl_handler() expects parameter 1 to be resource, double given in %s on line %d +Warning: xml_set_unparsed_entity_decl_handler() expects parameter 1 to be resource, float given in %s on line %d NULL Arg value Array diff --git a/ext/xml/xml.c b/ext/xml/xml.c index 84d4253b0f..bcded6203a 100644 --- a/ext/xml/xml.c +++ b/ext/xml/xml.c @@ -1,6 +1,6 @@ /* +----------------------------------------------------------------------+ - | PHP Version 5 | + | PHP Version 7 | +----------------------------------------------------------------------+ | Copyright (c) 1997-2015 The PHP Group | +----------------------------------------------------------------------+ @@ -62,6 +62,9 @@ ZEND_DECLARE_MODULE_GLOBALS(xml) /* {{{ dynamically loadable module stuff */ #ifdef COMPILE_DL_XML +#ifdef ZTS +ZEND_TSRMLS_CACHE_DEFINE(); +#endif ZEND_GET_MODULE(xml) #endif /* COMPILE_DL_XML */ /* }}} */ @@ -71,17 +74,17 @@ PHP_MINIT_FUNCTION(xml); PHP_MINFO_FUNCTION(xml); static PHP_GINIT_FUNCTION(xml); -static void xml_parser_dtor(zend_rsrc_list_entry *rsrc TSRMLS_DC); -static void xml_set_handler(zval **, zval **); +static void xml_parser_dtor(zend_resource *rsrc); +static void xml_set_handler(zval *, zval *); inline static unsigned short xml_encode_iso_8859_1(unsigned char); inline static char xml_decode_iso_8859_1(unsigned short); inline static unsigned short xml_encode_us_ascii(unsigned char); inline static char xml_decode_us_ascii(unsigned short); -static zval *xml_call_handler(xml_parser *, zval *, zend_function *, int, zval **); -static zval *_xml_xmlchar_zval(const XML_Char *, int, const XML_Char *); +static void xml_call_handler(xml_parser *, zval *, zend_function *, int, zval *, zval *); +static void _xml_xmlchar_zval(const XML_Char *, int, const XML_Char *, zval *); static int _xml_xmlcharlen(const XML_Char *); static void _xml_add_to_info(xml_parser *parser,char *name); -inline static char *_xml_decode_tag(xml_parser *parser, const char *tag); +inline static zend_string *_xml_decode_tag(xml_parser *parser, const char *tag); void _xml_startElementHandler(void *, const XML_Char *, const XML_Char **); void _xml_endElementHandler(void *, const XML_Char *); @@ -262,7 +265,7 @@ zend_module_entry xml_module_entry = { NULL, /* per-request startup function */ NULL, /* per-request shutdown function */ PHP_MINFO(xml), /* information function */ - NO_VERSION_YET, + PHP_XML_VERSION, PHP_MODULE_GLOBALS(xml), /* globals descriptor */ PHP_GINIT(xml), /* globals ctor */ NULL, /* globals dtor */ @@ -290,7 +293,10 @@ static int le_xml_parser; /* {{{ startup, shutdown and info functions */ static PHP_GINIT_FUNCTION(xml) { - xml_globals->default_encoding = "UTF-8"; +#if defined(COMPILE_DL_XML) && defined(ZTS) + ZEND_TSRMLS_CACHE_UPDATE(); +#endif + xml_globals->default_encoding = (XML_Char*)"UTF-8"; } static void *php_xml_malloc_wrapper(size_t sz) @@ -373,53 +379,22 @@ PHP_MINFO_FUNCTION(xml) /* }}} */ /* {{{ extension-internal functions */ -static zval *_xml_resource_zval(long value) -{ - zval *ret; - TSRMLS_FETCH(); - - MAKE_STD_ZVAL(ret); - - Z_TYPE_P(ret) = IS_RESOURCE; - Z_LVAL_P(ret) = value; - - zend_list_addref(value); - - return ret; -} - -static zval *_xml_string_zval(const char *str) -{ - zval *ret; - int len = strlen(str); - MAKE_STD_ZVAL(ret); - - Z_TYPE_P(ret) = IS_STRING; - Z_STRLEN_P(ret) = len; - Z_STRVAL_P(ret) = estrndup(str, len); - return ret; -} -static zval *_xml_xmlchar_zval(const XML_Char *s, int len, const XML_Char *encoding) +static void _xml_xmlchar_zval(const XML_Char *s, int len, const XML_Char *encoding, zval *ret) { - zval *ret; - MAKE_STD_ZVAL(ret); - if (s == NULL) { ZVAL_FALSE(ret); - return ret; + return; } if (len == 0) { len = _xml_xmlcharlen(s); } - Z_TYPE_P(ret) = IS_STRING; - Z_STRVAL_P(ret) = xml_utf8_decode(s, len, &Z_STRLEN_P(ret), encoding); - return ret; + ZVAL_STR(ret, xml_utf8_decode(s, len, encoding)); } /* }}} */ /* {{{ xml_parser_dtor() */ -static void xml_parser_dtor(zend_rsrc_list_entry *rsrc TSRMLS_DC) +static void xml_parser_dtor(zend_resource *rsrc) { xml_parser *parser = (xml_parser *)rsrc->ptr; @@ -432,43 +407,43 @@ static void xml_parser_dtor(zend_rsrc_list_entry *rsrc TSRMLS_DC) efree(parser->ltags[ inx ]); efree(parser->ltags); } - if (parser->startElementHandler) { + if (!Z_ISUNDEF(parser->startElementHandler)) { zval_ptr_dtor(&parser->startElementHandler); } - if (parser->endElementHandler) { + if (!Z_ISUNDEF(parser->endElementHandler)) { zval_ptr_dtor(&parser->endElementHandler); } - if (parser->characterDataHandler) { + if (!Z_ISUNDEF(parser->characterDataHandler)) { zval_ptr_dtor(&parser->characterDataHandler); } - if (parser->processingInstructionHandler) { + if (!Z_ISUNDEF(parser->processingInstructionHandler)) { zval_ptr_dtor(&parser->processingInstructionHandler); } - if (parser->defaultHandler) { + if (!Z_ISUNDEF(parser->defaultHandler)) { zval_ptr_dtor(&parser->defaultHandler); } - if (parser->unparsedEntityDeclHandler) { + if (!Z_ISUNDEF(parser->unparsedEntityDeclHandler)) { zval_ptr_dtor(&parser->unparsedEntityDeclHandler); } - if (parser->notationDeclHandler) { + if (!Z_ISUNDEF(parser->notationDeclHandler)) { zval_ptr_dtor(&parser->notationDeclHandler); } - if (parser->externalEntityRefHandler) { + if (!Z_ISUNDEF(parser->externalEntityRefHandler)) { zval_ptr_dtor(&parser->externalEntityRefHandler); } - if (parser->unknownEncodingHandler) { + if (!Z_ISUNDEF(parser->unknownEncodingHandler)) { zval_ptr_dtor(&parser->unknownEncodingHandler); } - if (parser->startNamespaceDeclHandler) { + if (!Z_ISUNDEF(parser->startNamespaceDeclHandler)) { zval_ptr_dtor(&parser->startNamespaceDeclHandler); } - if (parser->endNamespaceDeclHandler) { + if (!Z_ISUNDEF(parser->endNamespaceDeclHandler)) { zval_ptr_dtor(&parser->endNamespaceDeclHandler); } if (parser->baseURI) { efree(parser->baseURI); } - if (parser->object) { + if (!Z_ISUNDEF(parser->object)) { zval_ptr_dtor(&parser->object); } @@ -477,88 +452,65 @@ static void xml_parser_dtor(zend_rsrc_list_entry *rsrc TSRMLS_DC) /* }}} */ /* {{{ xml_set_handler() */ -static void xml_set_handler(zval **handler, zval **data) +static void xml_set_handler(zval *handler, zval *data) { /* If we have already a handler, release it */ - if (*handler) { + if (handler) { zval_ptr_dtor(handler); } /* IS_ARRAY might indicate that we're using array($obj, 'method') syntax */ - if (Z_TYPE_PP(data) != IS_ARRAY && Z_TYPE_PP(data) != IS_OBJECT) { - + if (Z_TYPE_P(data) != IS_ARRAY && Z_TYPE_P(data) != IS_OBJECT) { convert_to_string_ex(data); - if (Z_STRLEN_PP(data) == 0) { - *handler = NULL; + if (Z_STRLEN_P(data) == 0) { + ZVAL_UNDEF(handler); return; } } - zval_add_ref(data); - - *handler = *data; + ZVAL_COPY(handler, data); } /* }}} */ /* {{{ xml_call_handler() */ -static zval *xml_call_handler(xml_parser *parser, zval *handler, zend_function *function_ptr, int argc, zval **argv) +static void xml_call_handler(xml_parser *parser, zval *handler, zend_function *function_ptr, int argc, zval *argv, zval *retval) { int i; - TSRMLS_FETCH(); + ZVAL_UNDEF(retval); if (parser && handler && !EG(exception)) { - zval ***args; - zval *retval; int result; zend_fcall_info fci; - args = safe_emalloc(sizeof(zval **), argc, 0); - for (i = 0; i < argc; i++) { - args[i] = &argv[i]; - } - fci.size = sizeof(fci); fci.function_table = EG(function_table); - fci.function_name = handler; + ZVAL_COPY_VALUE(&fci.function_name, handler); fci.symbol_table = NULL; - fci.object_ptr = parser->object; - fci.retval_ptr_ptr = &retval; + fci.object = Z_OBJ(parser->object); + fci.retval = retval; fci.param_count = argc; - fci.params = args; + fci.params = argv; fci.no_separation = 0; /*fci.function_handler_cache = &function_ptr;*/ - result = zend_call_function(&fci, NULL TSRMLS_CC); + result = zend_call_function(&fci, NULL); if (result == FAILURE) { - zval **method; - zval **obj; + zval *method; + zval *obj; if (Z_TYPE_P(handler) == IS_STRING) { - php_error_docref(NULL TSRMLS_CC, E_WARNING, "Unable to call handler %s()", Z_STRVAL_P(handler)); - } else if (zend_hash_index_find(Z_ARRVAL_P(handler), 0, (void **) &obj) == SUCCESS && - zend_hash_index_find(Z_ARRVAL_P(handler), 1, (void **) &method) == SUCCESS && - Z_TYPE_PP(obj) == IS_OBJECT && - Z_TYPE_PP(method) == IS_STRING) { - php_error_docref(NULL TSRMLS_CC, E_WARNING, "Unable to call handler %s::%s()", Z_OBJCE_PP(obj)->name, Z_STRVAL_PP(method)); + php_error_docref(NULL, E_WARNING, "Unable to call handler %s()", Z_STRVAL_P(handler)); + } else if ((obj = zend_hash_index_find(Z_ARRVAL_P(handler), 0)) != NULL && + (method = zend_hash_index_find(Z_ARRVAL_P(handler), 1)) != NULL && + Z_TYPE_P(obj) == IS_OBJECT && + Z_TYPE_P(method) == IS_STRING) { + php_error_docref(NULL, E_WARNING, "Unable to call handler %s::%s()", ZSTR_VAL(Z_OBJCE_P(obj)->name), Z_STRVAL_P(method)); } else - php_error_docref(NULL TSRMLS_CC, E_WARNING, "Unable to call handler"); - } - - for (i = 0; i < argc; i++) { - zval_ptr_dtor(args[i]); - } - efree(args); - - if (result == FAILURE) { - return NULL; - } else { - return EG(exception) ? NULL : retval; - } - } else { - for (i = 0; i < argc; i++) { - zval_ptr_dtor(&argv[i]); + php_error_docref(NULL, E_WARNING, "Unable to call handler"); } - return NULL; + } + for (i = 0; i < argc; i++) { + zval_ptr_dtor(&argv[i]); } } /* }}} */ @@ -597,7 +549,7 @@ static xml_encoding *xml_get_encoding(const XML_Char *name) xml_encoding *enc = &xml_encodings[0]; while (enc && enc->name) { - if (strcasecmp(name, enc->name) == 0) { + if (strcasecmp((char *)name, (char *)enc->name) == 0) { return enc; } enc++; @@ -606,16 +558,15 @@ static xml_encoding *xml_get_encoding(const XML_Char *name) } /* }}} */ -/* {{{ xml_utf8_encode */ -PHPAPI char *xml_utf8_encode(const char *s, int len, int *newlen, const XML_Char *encoding) +/* {{{ xml_utf8_encode() */ +PHP_XML_API zend_string *xml_utf8_encode(const char *s, size_t len, const XML_Char *encoding) { - int pos = len; - char *newbuf; + size_t pos = len; + zend_string *str; unsigned int c; unsigned short (*encoder)(unsigned char) = NULL; xml_encoding *enc = xml_get_encoding(encoding); - *newlen = 0; if (enc) { encoder = enc->encoding_function; } else { @@ -625,65 +576,63 @@ PHPAPI char *xml_utf8_encode(const char *s, int len, int *newlen, const XML_Char if (encoder == NULL) { /* If no encoder function was specified, return the data as-is. */ - newbuf = emalloc(len + 1); - memcpy(newbuf, s, len); - *newlen = len; - newbuf[*newlen] = '\0'; - return newbuf; + str = zend_string_init(s, len, 0); + return str; } /* This is the theoretical max (will never get beyond len * 2 as long * as we are converting from single-byte characters, though) */ - newbuf = safe_emalloc(len, 4, 1); + str = zend_string_alloc(len * 4, 0); + ZSTR_LEN(str) = 0; while (pos > 0) { c = encoder ? encoder((unsigned char)(*s)) : (unsigned short)(*s); if (c < 0x80) { - newbuf[(*newlen)++] = (char) c; + ZSTR_VAL(str)[ZSTR_LEN(str)++] = (char) c; } else if (c < 0x800) { - newbuf[(*newlen)++] = (0xc0 | (c >> 6)); - newbuf[(*newlen)++] = (0x80 | (c & 0x3f)); + ZSTR_VAL(str)[ZSTR_LEN(str)++] = (0xc0 | (c >> 6)); + ZSTR_VAL(str)[ZSTR_LEN(str)++] = (0x80 | (c & 0x3f)); } else if (c < 0x10000) { - newbuf[(*newlen)++] = (0xe0 | (c >> 12)); - newbuf[(*newlen)++] = (0xc0 | ((c >> 6) & 0x3f)); - newbuf[(*newlen)++] = (0x80 | (c & 0x3f)); + ZSTR_VAL(str)[ZSTR_LEN(str)++] = (0xe0 | (c >> 12)); + ZSTR_VAL(str)[ZSTR_LEN(str)++] = (0xc0 | ((c >> 6) & 0x3f)); + ZSTR_VAL(str)[ZSTR_LEN(str)++] = (0x80 | (c & 0x3f)); } else if (c < 0x200000) { - newbuf[(*newlen)++] = (0xf0 | (c >> 18)); - newbuf[(*newlen)++] = (0xe0 | ((c >> 12) & 0x3f)); - newbuf[(*newlen)++] = (0xc0 | ((c >> 6) & 0x3f)); - newbuf[(*newlen)++] = (0x80 | (c & 0x3f)); + ZSTR_VAL(str)[ZSTR_LEN(str)++] = (0xf0 | (c >> 18)); + ZSTR_VAL(str)[ZSTR_LEN(str)++] = (0xe0 | ((c >> 12) & 0x3f)); + ZSTR_VAL(str)[ZSTR_LEN(str)++] = (0xc0 | ((c >> 6) & 0x3f)); + ZSTR_VAL(str)[ZSTR_LEN(str)++] = (0x80 | (c & 0x3f)); } pos--; s++; } - newbuf[*newlen] = 0; - newbuf = erealloc(newbuf, (*newlen)+1); - return newbuf; + ZSTR_VAL(str)[ZSTR_LEN(str)] = '\0'; + str = zend_string_truncate(str, ZSTR_LEN(str), 0); + return str; } /* }}} */ -/* {{{ xml_utf8_decode */ -PHPAPI char *xml_utf8_decode(const XML_Char *s, int len, int *newlen, const XML_Char *encoding) +/* {{{ xml_utf8_decode() */ +PHP_XML_API zend_string *xml_utf8_decode(const XML_Char *s, size_t len, const XML_Char *encoding) { size_t pos = 0; - char *newbuf = emalloc(len + 1); unsigned int c; char (*decoder)(unsigned short) = NULL; xml_encoding *enc = xml_get_encoding(encoding); + zend_string *str; - *newlen = 0; if (enc) { decoder = enc->decoding_function; } + if (decoder == NULL) { /* If the target encoding was unknown, or no decoder function * was specified, return the UTF-8-encoded data as-is. */ - memcpy(newbuf, s, len); - *newlen = len; - newbuf[*newlen] = '\0'; - return newbuf; + str = zend_string_init((char *)s, len, 0); + return str; } - while (pos < (size_t)len) { + str = zend_string_alloc(len, 0); + ZSTR_LEN(str) = 0; + while (pos < len) { int status = FAILURE; c = php_next_utf8_char((const unsigned char*)s, (size_t) len, &pos, &status); @@ -691,14 +640,14 @@ PHPAPI char *xml_utf8_decode(const XML_Char *s, int len, int *newlen, const XML_ c = '?'; } - newbuf[*newlen] = decoder ? decoder(c) : c; - ++*newlen; + ZSTR_VAL(str)[ZSTR_LEN(str)++] = decoder ? decoder(c) : c; } - if (*newlen < len) { - newbuf = erealloc(newbuf, *newlen + 1); + ZSTR_VAL(str)[ZSTR_LEN(str)] = '\0'; + if (ZSTR_LEN(str) < len) { + str = zend_string_truncate(str, ZSTR_LEN(str), 0); } - newbuf[*newlen] = '\0'; - return newbuf; + + return str; } /* }}} */ @@ -716,7 +665,7 @@ static int _xml_xmlcharlen(const XML_Char *s) /* }}} */ /* {{{ _xml_zval_strdup() */ -PHPAPI char *_xml_zval_strdup(zval *val) +PHP_XML_API char *_xml_zval_strdup(zval *val) { if (Z_TYPE_P(val) == IS_STRING) { char *buf = emalloc(Z_STRLEN_P(val) + 1); @@ -728,42 +677,39 @@ PHPAPI char *_xml_zval_strdup(zval *val) } /* }}} */ -/* {{{ _xml_add_to_info */ +/* {{{ _xml_add_to_info() */ static void _xml_add_to_info(xml_parser *parser,char *name) { - zval **element, *values; + zval *element; - if (! parser->info) { + if (Z_ISUNDEF(parser->info)) { return; } - if (zend_hash_find(Z_ARRVAL_P(parser->info),name,strlen(name) + 1,(void **) &element) == FAILURE) { - MAKE_STD_ZVAL(values); - - array_init(values); - - zend_hash_update(Z_ARRVAL_P(parser->info), name, strlen(name)+1, (void *) &values, sizeof(zval*), (void **) &element); + if ((element = zend_hash_str_find(Z_ARRVAL(parser->info), name, strlen(name))) == NULL) { + zval values; + array_init(&values); + element = zend_hash_str_update(Z_ARRVAL(parser->info), name, strlen(name), &values); } - add_next_index_long(*element,parser->curtag); + add_next_index_long(element, parser->curtag); parser->curtag++; } /* }}} */ /* {{{ _xml_decode_tag() */ -static char *_xml_decode_tag(xml_parser *parser, const char *tag) +static zend_string *_xml_decode_tag(xml_parser *parser, const char *tag) { - char *newstr; - int out_len; + zend_string *str; - newstr = xml_utf8_decode(tag, strlen(tag), &out_len, parser->target_encoding); + str = xml_utf8_decode((const XML_Char *)tag, strlen(tag), parser->target_encoding); if (parser->case_folding) { - php_strtoupper(newstr, out_len); + php_strtoupper(ZSTR_VAL(str), ZSTR_LEN(str)); } - return newstr; + return str; } /* }}} */ @@ -772,86 +718,84 @@ void _xml_startElementHandler(void *userData, const XML_Char *name, const XML_Ch { xml_parser *parser = (xml_parser *)userData; const char **attrs = (const char **) attributes; - char *tag_name; - char *att, *val; - int val_len; - zval *retval, *args[3]; + zend_string *att, *tag_name, *val; + zval retval, args[3]; if (parser) { parser->level++; - tag_name = _xml_decode_tag(parser, name); + tag_name = _xml_decode_tag(parser, (const char *)name); - if (parser->startElementHandler) { - args[0] = _xml_resource_zval(parser->index); - args[1] = _xml_string_zval(((char *) tag_name) + parser->toffset); - MAKE_STD_ZVAL(args[2]); - array_init(args[2]); + if (!Z_ISUNDEF(parser->startElementHandler)) { + ZVAL_COPY(&args[0], &parser->index); + ZVAL_STRING(&args[1], ZSTR_VAL(tag_name) + parser->toffset); + array_init(&args[2]); while (attributes && *attributes) { - att = _xml_decode_tag(parser, attributes[0]); - val = xml_utf8_decode(attributes[1], strlen(attributes[1]), &val_len, parser->target_encoding); + zval tmp; - add_assoc_stringl(args[2], att, val, val_len, 0); + att = _xml_decode_tag(parser, (const char *)attributes[0]); + val = xml_utf8_decode(attributes[1], strlen((char *)attributes[1]), parser->target_encoding); + + ZVAL_STR(&tmp, val); + zend_symtable_update(Z_ARRVAL(args[2]), att, &tmp); attributes += 2; - efree(att); + zend_string_release(att); } - if ((retval = xml_call_handler(parser, parser->startElementHandler, parser->startElementPtr, 3, args))) { - zval_ptr_dtor(&retval); - } + xml_call_handler(parser, &parser->startElementHandler, parser->startElementPtr, 3, args, &retval); + zval_ptr_dtor(&retval); } - if (parser->data) { + if (!Z_ISUNDEF(parser->data)) { if (parser->level <= XML_MAXLEVEL) { - zval *tag, *atr; + zval tag, atr; int atcnt = 0; - MAKE_STD_ZVAL(tag); - MAKE_STD_ZVAL(atr); + array_init(&tag); + array_init(&atr); - array_init(tag); - array_init(atr); + _xml_add_to_info(parser, ZSTR_VAL(tag_name) + parser->toffset); - _xml_add_to_info(parser,((char *) tag_name) + parser->toffset); + add_assoc_string(&tag, "tag", ZSTR_VAL(tag_name) + parser->toffset); /* cast to avoid gcc-warning */ + add_assoc_string(&tag, "type", "open"); + add_assoc_long(&tag, "level", parser->level); - add_assoc_string(tag,"tag",((char *) tag_name) + parser->toffset,1); /* cast to avoid gcc-warning */ - add_assoc_string(tag,"type","open",1); - add_assoc_long(tag,"level",parser->level); - - parser->ltags[parser->level-1] = estrdup(tag_name); + parser->ltags[parser->level-1] = estrdup(ZSTR_VAL(tag_name)); parser->lastwasopen = 1; attributes = (const XML_Char **) attrs; while (attributes && *attributes) { - att = _xml_decode_tag(parser, attributes[0]); - val = xml_utf8_decode(attributes[1], strlen(attributes[1]), &val_len, parser->target_encoding); + zval tmp; + + att = _xml_decode_tag(parser, (const char *)attributes[0]); + val = xml_utf8_decode(attributes[1], strlen((char *)attributes[1]), parser->target_encoding); - add_assoc_stringl(atr,att,val,val_len,0); + ZVAL_STR(&tmp, val); + zend_symtable_update(Z_ARRVAL(atr), att, &tmp); atcnt++; attributes += 2; - efree(att); + zend_string_release(att); } if (atcnt) { - zend_hash_add(Z_ARRVAL_P(tag),"attributes",sizeof("attributes"),&atr,sizeof(zval*),NULL); + zend_hash_str_add(Z_ARRVAL(tag), "attributes", sizeof("attributes") - 1, &atr); } else { zval_ptr_dtor(&atr); } - zend_hash_next_index_insert(Z_ARRVAL_P(parser->data),&tag,sizeof(zval*),(void *) &parser->ctag); + parser->ctag = zend_hash_next_index_insert(Z_ARRVAL(parser->data), &tag); } else if (parser->level == (XML_MAXLEVEL + 1)) { - TSRMLS_FETCH(); - php_error_docref(NULL TSRMLS_CC, E_WARNING, "Maximum depth exceeded - Results truncated"); + php_error_docref(NULL, E_WARNING, "Maximum depth exceeded - Results truncated"); } } - efree(tag_name); + zend_string_release(tag_name); } } /* }}} */ @@ -860,45 +804,42 @@ void _xml_startElementHandler(void *userData, const XML_Char *name, const XML_Ch void _xml_endElementHandler(void *userData, const XML_Char *name) { xml_parser *parser = (xml_parser *)userData; - char *tag_name; + zend_string *tag_name; if (parser) { - zval *retval, *args[2]; + zval retval, args[2]; - tag_name = _xml_decode_tag(parser, name); + tag_name = _xml_decode_tag(parser, (const char *)name); - if (parser->endElementHandler) { - args[0] = _xml_resource_zval(parser->index); - args[1] = _xml_string_zval(((char *) tag_name) + parser->toffset); + if (!Z_ISUNDEF(parser->endElementHandler)) { + ZVAL_COPY(&args[0], &parser->index); + ZVAL_STRING(&args[1], ZSTR_VAL(tag_name) + parser->toffset); - if ((retval = xml_call_handler(parser, parser->endElementHandler, parser->endElementPtr, 2, args))) { - zval_ptr_dtor(&retval); - } + xml_call_handler(parser, &parser->endElementHandler, parser->endElementPtr, 2, args, &retval); + zval_ptr_dtor(&retval); } - if (parser->data) { - zval *tag; + if (!Z_ISUNDEF(parser->data)) { + zval tag; if (parser->lastwasopen) { - add_assoc_string(*(parser->ctag),"type","complete",1); + add_assoc_string(parser->ctag, "type", "complete"); } else { - MAKE_STD_ZVAL(tag); - - array_init(tag); + array_init(&tag); - _xml_add_to_info(parser,((char *) tag_name) + parser->toffset); + _xml_add_to_info(parser, ZSTR_VAL(tag_name) + parser->toffset); - add_assoc_string(tag,"tag",((char *) tag_name) + parser->toffset,1); /* cast to avoid gcc-warning */ - add_assoc_string(tag,"type","close",1); - add_assoc_long(tag,"level",parser->level); + add_assoc_string(&tag, "tag", ZSTR_VAL(tag_name) + parser->toffset); /* cast to avoid gcc-warning */ + add_assoc_string(&tag, "type", "close"); + add_assoc_long(&tag, "level", parser->level); - zend_hash_next_index_insert(Z_ARRVAL_P(parser->data),&tag,sizeof(zval*),NULL); + zend_hash_next_index_insert(Z_ARRVAL(parser->data), &tag); } parser->lastwasopen = 0; } - efree(tag_name); + zend_string_release(tag_name); if ((parser->ltags) && (parser->level <= XML_MAXLEVEL)) { efree(parser->ltags[parser->level-1]); @@ -915,33 +856,30 @@ void _xml_characterDataHandler(void *userData, const XML_Char *s, int len) xml_parser *parser = (xml_parser *)userData; if (parser) { - zval *retval, *args[2]; + zval retval, args[2]; - if (parser->characterDataHandler) { - args[0] = _xml_resource_zval(parser->index); - args[1] = _xml_xmlchar_zval(s, len, parser->target_encoding); - if ((retval = xml_call_handler(parser, parser->characterDataHandler, parser->characterDataPtr, 2, args))) { - zval_ptr_dtor(&retval); - } + if (!Z_ISUNDEF(parser->characterDataHandler)) { + ZVAL_COPY(&args[0], &parser->index); + _xml_xmlchar_zval(s, len, parser->target_encoding, &args[1]); + xml_call_handler(parser, &parser->characterDataHandler, parser->characterDataPtr, 2, args, &retval); + zval_ptr_dtor(&retval); } - if (parser->data) { + if (!Z_ISUNDEF(parser->data)) { int i; int doprint = 0; - - char *decoded_value; - int decoded_len; - - decoded_value = xml_utf8_decode(s,len,&decoded_len,parser->target_encoding); - for (i = 0; i < decoded_len; i++) { - switch (decoded_value[i]) { - case ' ': - case '\t': - case '\n': - continue; - default: - doprint = 1; - break; + zend_string *decoded_value; + + decoded_value = xml_utf8_decode(s, len, parser->target_encoding); + for (i = 0; i < ZSTR_LEN(decoded_value); i++) { + switch (ZSTR_VAL(decoded_value)[i]) { + case ' ': + case '\t': + case '\n': + continue; + default: + doprint = 1; + break; } if (doprint) { break; @@ -949,61 +887,56 @@ void _xml_characterDataHandler(void *userData, const XML_Char *s, int len) } if (doprint || (! parser->skipwhite)) { if (parser->lastwasopen) { - zval **myval; + zval *myval; /* check if the current tag already has a value - if yes append to that! */ - if (zend_hash_find(Z_ARRVAL_PP(parser->ctag),"value",sizeof("value"),(void **) &myval) == SUCCESS) { - int newlen = Z_STRLEN_PP(myval) + decoded_len; - Z_STRVAL_PP(myval) = erealloc(Z_STRVAL_PP(myval),newlen+1); - strncpy(Z_STRVAL_PP(myval) + Z_STRLEN_PP(myval), decoded_value, decoded_len + 1); - Z_STRLEN_PP(myval) += decoded_len; - efree(decoded_value); + if ((myval = zend_hash_str_find(Z_ARRVAL_P(parser->ctag), "value", sizeof("value") - 1))) { + int newlen = Z_STRLEN_P(myval) + ZSTR_LEN(decoded_value); + Z_STR_P(myval) = zend_string_extend(Z_STR_P(myval), newlen, 0); + strncpy(Z_STRVAL_P(myval) + Z_STRLEN_P(myval) - ZSTR_LEN(decoded_value), + ZSTR_VAL(decoded_value), ZSTR_LEN(decoded_value) + 1); + zend_string_release(decoded_value); } else { - add_assoc_string(*(parser->ctag),"value",decoded_value,0); + add_assoc_str(parser->ctag, "value", decoded_value); } } else { - zval *tag; - zval **curtag, **mytype, **myval; - HashPosition hpos=NULL; - - zend_hash_internal_pointer_end_ex(Z_ARRVAL_P(parser->data), &hpos); - - if (hpos && (zend_hash_get_current_data_ex(Z_ARRVAL_P(parser->data), (void **) &curtag, &hpos) == SUCCESS)) { - if (zend_hash_find(Z_ARRVAL_PP(curtag),"type",sizeof("type"),(void **) &mytype) == SUCCESS) { - if (!strcmp(Z_STRVAL_PP(mytype), "cdata")) { - if (zend_hash_find(Z_ARRVAL_PP(curtag),"value",sizeof("value"),(void **) &myval) == SUCCESS) { - int newlen = Z_STRLEN_PP(myval) + decoded_len; - Z_STRVAL_PP(myval) = erealloc(Z_STRVAL_PP(myval),newlen+1); - strncpy(Z_STRVAL_PP(myval) + Z_STRLEN_PP(myval), decoded_value, decoded_len + 1); - Z_STRLEN_PP(myval) += decoded_len; - efree(decoded_value); + zval tag; + zval *curtag, *mytype, *myval; + + ZEND_HASH_REVERSE_FOREACH_VAL(Z_ARRVAL(parser->data), curtag) { + if ((mytype = zend_hash_str_find(Z_ARRVAL_P(curtag),"type", sizeof("type") - 1))) { + if (!strcmp(Z_STRVAL_P(mytype), "cdata")) { + if ((myval = zend_hash_str_find(Z_ARRVAL_P(curtag), "value", sizeof("value") - 1))) { + int newlen = Z_STRLEN_P(myval) + ZSTR_LEN(decoded_value); + Z_STR_P(myval) = zend_string_extend(Z_STR_P(myval), newlen, 0); + strncpy(Z_STRVAL_P(myval) + Z_STRLEN_P(myval) - ZSTR_LEN(decoded_value), + ZSTR_VAL(decoded_value), ZSTR_LEN(decoded_value) + 1); + zend_string_release(decoded_value); return; } } } - } + break; + } ZEND_HASH_FOREACH_END(); if (parser->level <= XML_MAXLEVEL) { - MAKE_STD_ZVAL(tag); - - array_init(tag); + array_init(&tag); _xml_add_to_info(parser,parser->ltags[parser->level-1] + parser->toffset); - add_assoc_string(tag,"tag",parser->ltags[parser->level-1] + parser->toffset,1); - add_assoc_string(tag,"value",decoded_value,0); - add_assoc_string(tag,"type","cdata",1); - add_assoc_long(tag,"level",parser->level); + add_assoc_string(&tag, "tag", parser->ltags[parser->level-1] + parser->toffset); + add_assoc_str(&tag, "value", decoded_value); + add_assoc_string(&tag, "type", "cdata"); + add_assoc_long(&tag, "level", parser->level); - zend_hash_next_index_insert(Z_ARRVAL_P(parser->data),&tag,sizeof(zval*),NULL); + zend_hash_next_index_insert(Z_ARRVAL(parser->data), &tag); } else if (parser->level == (XML_MAXLEVEL + 1)) { - TSRMLS_FETCH(); - php_error_docref(NULL TSRMLS_CC, E_WARNING, "Maximum depth exceeded - Results truncated"); + php_error_docref(NULL, E_WARNING, "Maximum depth exceeded - Results truncated"); } } } else { - efree(decoded_value); + zend_string_release(decoded_value); } } } @@ -1015,15 +948,14 @@ void _xml_processingInstructionHandler(void *userData, const XML_Char *target, c { xml_parser *parser = (xml_parser *)userData; - if (parser && parser->processingInstructionHandler) { - zval *retval, *args[3]; + if (parser && !Z_ISUNDEF(parser->processingInstructionHandler)) { + zval retval, args[3]; - args[0] = _xml_resource_zval(parser->index); - args[1] = _xml_xmlchar_zval(target, 0, parser->target_encoding); - args[2] = _xml_xmlchar_zval(data, 0, parser->target_encoding); - if ((retval = xml_call_handler(parser, parser->processingInstructionHandler, parser->processingInstructionPtr, 3, args))) { - zval_ptr_dtor(&retval); - } + ZVAL_COPY(&args[0], &parser->index); + _xml_xmlchar_zval(target, 0, parser->target_encoding, &args[1]); + _xml_xmlchar_zval(data, 0, parser->target_encoding, &args[2]); + xml_call_handler(parser, &parser->processingInstructionHandler, parser->processingInstructionPtr, 3, args, &retval); + zval_ptr_dtor(&retval); } } /* }}} */ @@ -1033,14 +965,13 @@ void _xml_defaultHandler(void *userData, const XML_Char *s, int len) { xml_parser *parser = (xml_parser *)userData; - if (parser && parser->defaultHandler) { - zval *retval, *args[2]; + if (parser && !Z_ISUNDEF(parser->defaultHandler)) { + zval retval, args[2]; - args[0] = _xml_resource_zval(parser->index); - args[1] = _xml_xmlchar_zval(s, len, parser->target_encoding); - if ((retval = xml_call_handler(parser, parser->defaultHandler, parser->defaultPtr, 2, args))) { - zval_ptr_dtor(&retval); - } + ZVAL_COPY(&args[0], &parser->index); + _xml_xmlchar_zval(s, len, parser->target_encoding, &args[1]); + xml_call_handler(parser, &parser->defaultHandler, parser->defaultPtr, 2, args, &retval); + zval_ptr_dtor(&retval); } } /* }}} */ @@ -1055,18 +986,17 @@ void _xml_unparsedEntityDeclHandler(void *userData, { xml_parser *parser = (xml_parser *)userData; - if (parser && parser->unparsedEntityDeclHandler) { - zval *retval, *args[6]; + if (parser && !Z_ISUNDEF(parser->unparsedEntityDeclHandler)) { + zval retval, args[6]; - args[0] = _xml_resource_zval(parser->index); - args[1] = _xml_xmlchar_zval(entityName, 0, parser->target_encoding); - args[2] = _xml_xmlchar_zval(base, 0, parser->target_encoding); - args[3] = _xml_xmlchar_zval(systemId, 0, parser->target_encoding); - args[4] = _xml_xmlchar_zval(publicId, 0, parser->target_encoding); - args[5] = _xml_xmlchar_zval(notationName, 0, parser->target_encoding); - if ((retval = xml_call_handler(parser, parser->unparsedEntityDeclHandler, parser->unparsedEntityDeclPtr, 6, args))) { - zval_ptr_dtor(&retval); - } + ZVAL_COPY(&args[0], &parser->index); + _xml_xmlchar_zval(entityName, 0, parser->target_encoding, &args[1]); + _xml_xmlchar_zval(base, 0, parser->target_encoding, &args[2]); + _xml_xmlchar_zval(systemId, 0, parser->target_encoding, &args[3]); + _xml_xmlchar_zval(publicId, 0, parser->target_encoding, &args[4]); + _xml_xmlchar_zval(notationName, 0, parser->target_encoding, &args[5]); + xml_call_handler(parser, &parser->unparsedEntityDeclHandler, parser->unparsedEntityDeclPtr, 6, args, &retval); + zval_ptr_dtor(&retval); } } /* }}} */ @@ -1080,17 +1010,16 @@ void _xml_notationDeclHandler(void *userData, { xml_parser *parser = (xml_parser *)userData; - if (parser && parser->notationDeclHandler) { - zval *retval, *args[5]; + if (parser && !Z_ISUNDEF(parser->notationDeclHandler)) { + zval retval, args[5]; - args[0] = _xml_resource_zval(parser->index); - args[1] = _xml_xmlchar_zval(notationName, 0, parser->target_encoding); - args[2] = _xml_xmlchar_zval(base, 0, parser->target_encoding); - args[3] = _xml_xmlchar_zval(systemId, 0, parser->target_encoding); - args[4] = _xml_xmlchar_zval(publicId, 0, parser->target_encoding); - if ((retval = xml_call_handler(parser, parser->notationDeclHandler, parser->notationDeclPtr, 5, args))) { - zval_ptr_dtor(&retval); - } + ZVAL_COPY(&args[0], &parser->index); + _xml_xmlchar_zval(notationName, 0, parser->target_encoding, &args[1]); + _xml_xmlchar_zval(base, 0, parser->target_encoding, &args[2]); + _xml_xmlchar_zval(systemId, 0, parser->target_encoding, &args[3]); + _xml_xmlchar_zval(publicId, 0, parser->target_encoding, &args[4]); + xml_call_handler(parser, &parser->notationDeclHandler, parser->notationDeclPtr, 5, args, &retval); + zval_ptr_dtor(&retval); } } /* }}} */ @@ -1105,18 +1034,18 @@ int _xml_externalEntityRefHandler(XML_Parser parserPtr, xml_parser *parser = XML_GetUserData(parserPtr); int ret = 0; /* abort if no handler is set (should be configurable?) */ - if (parser && parser->externalEntityRefHandler) { - zval *retval, *args[5]; - - args[0] = _xml_resource_zval(parser->index); - args[1] = _xml_xmlchar_zval(openEntityNames, 0, parser->target_encoding); - args[2] = _xml_xmlchar_zval(base, 0, parser->target_encoding); - args[3] = _xml_xmlchar_zval(systemId, 0, parser->target_encoding); - args[4] = _xml_xmlchar_zval(publicId, 0, parser->target_encoding); - if ((retval = xml_call_handler(parser, parser->externalEntityRefHandler, parser->externalEntityRefPtr, 5, args))) { - convert_to_long(retval); - ret = Z_LVAL_P(retval); - efree(retval); + if (parser && !Z_ISUNDEF(parser->externalEntityRefHandler)) { + zval retval, args[5]; + + ZVAL_COPY(&args[0], &parser->index); + _xml_xmlchar_zval(openEntityNames, 0, parser->target_encoding, &args[1]); + _xml_xmlchar_zval(base, 0, parser->target_encoding, &args[2]); + _xml_xmlchar_zval(systemId, 0, parser->target_encoding, &args[3]); + _xml_xmlchar_zval(publicId, 0, parser->target_encoding, &args[4]); + xml_call_handler(parser, &parser->externalEntityRefHandler, parser->externalEntityRefPtr, 5, args, &retval); + if (!Z_ISUNDEF(retval)) { + convert_to_long(&retval); + ret = Z_LVAL(retval); } else { ret = 0; } @@ -1130,15 +1059,14 @@ void _xml_startNamespaceDeclHandler(void *userData,const XML_Char *prefix, const { xml_parser *parser = (xml_parser *)userData; - if (parser && parser->startNamespaceDeclHandler) { - zval *retval, *args[3]; + if (parser && !Z_ISUNDEF(parser->startNamespaceDeclHandler)) { + zval retval, args[3]; - args[0] = _xml_resource_zval(parser->index); - args[1] = _xml_xmlchar_zval(prefix, 0, parser->target_encoding); - args[2] = _xml_xmlchar_zval(uri, 0, parser->target_encoding); - if ((retval = xml_call_handler(parser, parser->startNamespaceDeclHandler, parser->startNamespaceDeclPtr, 3, args))) { - zval_ptr_dtor(&retval); - } + ZVAL_COPY(&args[0], &parser->index); + _xml_xmlchar_zval(prefix, 0, parser->target_encoding, &args[1]); + _xml_xmlchar_zval(uri, 0, parser->target_encoding, &args[2]); + xml_call_handler(parser, &parser->startNamespaceDeclHandler, parser->startNamespaceDeclPtr, 3, args, &retval); + zval_ptr_dtor(&retval); } } /* }}} */ @@ -1148,14 +1076,13 @@ void _xml_endNamespaceDeclHandler(void *userData, const XML_Char *prefix) { xml_parser *parser = (xml_parser *)userData; - if (parser && parser->endNamespaceDeclHandler) { - zval *retval, *args[2]; + if (parser && !Z_ISUNDEF(parser->endNamespaceDeclHandler)) { + zval retval, args[2]; - args[0] = _xml_resource_zval(parser->index); - args[1] = _xml_xmlchar_zval(prefix, 0, parser->target_encoding); - if ((retval = xml_call_handler(parser, parser->endNamespaceDeclHandler, parser->endNamespaceDeclPtr, 2, args))) { - zval_ptr_dtor(&retval); - } + ZVAL_COPY(&args[0], &parser->index); + _xml_xmlchar_zval(prefix, 0, parser->target_encoding, &args[1]); + xml_call_handler(parser, &parser->endNamespaceDeclHandler, parser->endNamespaceDeclPtr, 2, args, &retval); + zval_ptr_dtor(&retval); } } /* }}} */ @@ -1168,14 +1095,14 @@ static void php_xml_parser_create_impl(INTERNAL_FUNCTION_PARAMETERS, int ns_supp int auto_detect = 0; char *encoding_param = NULL; - int encoding_param_len = 0; + size_t encoding_param_len = 0; char *ns_param = NULL; - int ns_param_len = 0; + size_t ns_param_len = 0; XML_Char *encoding; - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, (ns_support ? "|ss": "|s"), &encoding_param, &encoding_param_len, &ns_param, &ns_param_len) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), (ns_support ? "|ss": "|s"), &encoding_param, &encoding_param_len, &ns_param, &ns_param_len) == FAILURE) { RETURN_FALSE; } @@ -1187,13 +1114,13 @@ static void php_xml_parser_create_impl(INTERNAL_FUNCTION_PARAMETERS, int ns_supp encoding = XML(default_encoding); auto_detect = 1; } else if (strcasecmp(encoding_param, "ISO-8859-1") == 0) { - encoding = "ISO-8859-1"; + encoding = (XML_Char*)"ISO-8859-1"; } else if (strcasecmp(encoding_param, "UTF-8") == 0) { - encoding = "UTF-8"; + encoding = (XML_Char*)"UTF-8"; } else if (strcasecmp(encoding_param, "US-ASCII") == 0) { - encoding = "US-ASCII"; + encoding = (XML_Char*)"US-ASCII"; } else { - php_error_docref(NULL TSRMLS_CC, E_WARNING, "unsupported source encoding \"%s\"", encoding_param); + php_error_docref(NULL, E_WARNING, "unsupported source encoding \"%s\"", encoding_param); RETURN_FALSE; } } else { @@ -1206,17 +1133,16 @@ static void php_xml_parser_create_impl(INTERNAL_FUNCTION_PARAMETERS, int ns_supp parser = ecalloc(1, sizeof(xml_parser)); parser->parser = XML_ParserCreate_MM((auto_detect ? NULL : encoding), - &php_xml_mem_hdlrs, ns_param); + &php_xml_mem_hdlrs, (XML_Char*)ns_param); parser->target_encoding = encoding; parser->case_folding = 1; - parser->object = NULL; parser->isparsing = 0; XML_SetUserData(parser->parser, parser); - ZEND_REGISTER_RESOURCE(return_value, parser,le_xml_parser); - parser->index = Z_LVAL_P(return_value); + RETVAL_RES(zend_register_resource(parser, le_xml_parser)); + ZVAL_COPY(&parser->index, return_value); } /* }}} */ @@ -1243,14 +1169,16 @@ PHP_FUNCTION(xml_set_object) xml_parser *parser; zval *pind, *mythis; - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "ro", &pind, &mythis) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "ro/", &pind, &mythis) == FAILURE) { return; } - ZEND_FETCH_RESOURCE(parser,xml_parser *, &pind, -1, "XML Parser", le_xml_parser); + if ((parser = (xml_parser *)zend_fetch_resource(Z_RES_P(pind), "XML Parser", le_xml_parser)) == NULL) { + RETURN_FALSE; + } /* please leave this commented - or ask thies@thieso.net before doing it (again) */ - if (parser->object) { + if (!Z_ISUNDEF(parser->object)) { zval_ptr_dtor(&parser->object); } @@ -1259,8 +1187,7 @@ PHP_FUNCTION(xml_set_object) zval_add_ref(&parser->object); #endif */ - ALLOC_ZVAL(parser->object); - MAKE_COPY_ZVAL(&mythis, parser->object); + ZVAL_COPY(&parser->object, mythis); RETVAL_TRUE; } @@ -1271,13 +1198,15 @@ PHP_FUNCTION(xml_set_object) PHP_FUNCTION(xml_set_element_handler) { xml_parser *parser; - zval *pind, **shdl, **ehdl; + zval *pind, *shdl, *ehdl; - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "rZZ", &pind, &shdl, &ehdl) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "rzz", &pind, &shdl, &ehdl) == FAILURE) { return; } - ZEND_FETCH_RESOURCE(parser,xml_parser *, &pind, -1, "XML Parser", le_xml_parser); + if ((parser = (xml_parser *)zend_fetch_resource(Z_RES_P(pind), "XML Parser", le_xml_parser)) == NULL) { + RETURN_FALSE; + } xml_set_handler(&parser->startElementHandler, shdl); xml_set_handler(&parser->endElementHandler, ehdl); @@ -1291,13 +1220,15 @@ PHP_FUNCTION(xml_set_element_handler) PHP_FUNCTION(xml_set_character_data_handler) { xml_parser *parser; - zval *pind, **hdl; + zval *pind, *hdl; - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "rZ", &pind, &hdl) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "rz", &pind, &hdl) == FAILURE) { return; } - ZEND_FETCH_RESOURCE(parser,xml_parser *, &pind, -1, "XML Parser", le_xml_parser); + if ((parser = (xml_parser *)zend_fetch_resource(Z_RES_P(pind), "XML Parser", le_xml_parser)) == NULL) { + RETURN_FALSE; + } xml_set_handler(&parser->characterDataHandler, hdl); XML_SetCharacterDataHandler(parser->parser, _xml_characterDataHandler); @@ -1310,13 +1241,15 @@ PHP_FUNCTION(xml_set_character_data_handler) PHP_FUNCTION(xml_set_processing_instruction_handler) { xml_parser *parser; - zval *pind, **hdl; + zval *pind, *hdl; - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "rZ", &pind, &hdl) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "rz", &pind, &hdl) == FAILURE) { return; } - ZEND_FETCH_RESOURCE(parser,xml_parser *, &pind, -1, "XML Parser", le_xml_parser); + if ((parser = (xml_parser *)zend_fetch_resource(Z_RES_P(pind), "XML Parser", le_xml_parser)) == NULL) { + RETURN_FALSE; + } xml_set_handler(&parser->processingInstructionHandler, hdl); XML_SetProcessingInstructionHandler(parser->parser, _xml_processingInstructionHandler); @@ -1329,12 +1262,15 @@ PHP_FUNCTION(xml_set_processing_instruction_handler) PHP_FUNCTION(xml_set_default_handler) { xml_parser *parser; - zval *pind, **hdl; + zval *pind, *hdl; - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "rZ", &pind, &hdl) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "rz", &pind, &hdl) == FAILURE) { return; } - ZEND_FETCH_RESOURCE(parser,xml_parser *, &pind, -1, "XML Parser", le_xml_parser); + + if ((parser = (xml_parser *)zend_fetch_resource(Z_RES_P(pind), "XML Parser", le_xml_parser)) == NULL) { + RETURN_FALSE; + } xml_set_handler(&parser->defaultHandler, hdl); XML_SetDefaultHandler(parser->parser, _xml_defaultHandler); @@ -1347,13 +1283,15 @@ PHP_FUNCTION(xml_set_default_handler) PHP_FUNCTION(xml_set_unparsed_entity_decl_handler) { xml_parser *parser; - zval *pind, **hdl; + zval *pind, *hdl; - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "rZ", &pind, &hdl) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "rz", &pind, &hdl) == FAILURE) { return; } - ZEND_FETCH_RESOURCE(parser,xml_parser *, &pind, -1, "XML Parser", le_xml_parser); + if ((parser = (xml_parser *)zend_fetch_resource(Z_RES_P(pind), "XML Parser", le_xml_parser)) == NULL) { + RETURN_FALSE; + } xml_set_handler(&parser->unparsedEntityDeclHandler, hdl); XML_SetUnparsedEntityDeclHandler(parser->parser, _xml_unparsedEntityDeclHandler); @@ -1366,12 +1304,15 @@ PHP_FUNCTION(xml_set_unparsed_entity_decl_handler) PHP_FUNCTION(xml_set_notation_decl_handler) { xml_parser *parser; - zval *pind, **hdl; + zval *pind, *hdl; - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "rZ", &pind, &hdl) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "rz", &pind, &hdl) == FAILURE) { return; } - ZEND_FETCH_RESOURCE(parser,xml_parser *, &pind, -1, "XML Parser", le_xml_parser); + + if ((parser = (xml_parser *)zend_fetch_resource(Z_RES_P(pind), "XML Parser", le_xml_parser)) == NULL) { + RETURN_FALSE; + } xml_set_handler(&parser->notationDeclHandler, hdl); XML_SetNotationDeclHandler(parser->parser, _xml_notationDeclHandler); @@ -1384,12 +1325,15 @@ PHP_FUNCTION(xml_set_notation_decl_handler) PHP_FUNCTION(xml_set_external_entity_ref_handler) { xml_parser *parser; - zval *pind, **hdl; + zval *pind, *hdl; - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "rZ", &pind, &hdl) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "rz", &pind, &hdl) == FAILURE) { return; } - ZEND_FETCH_RESOURCE(parser,xml_parser *, &pind, -1, "XML Parser", le_xml_parser); + + if ((parser = (xml_parser *)zend_fetch_resource(Z_RES_P(pind), "XML Parser", le_xml_parser)) == NULL) { + RETURN_FALSE; + } xml_set_handler(&parser->externalEntityRefHandler, hdl); XML_SetExternalEntityRefHandler(parser->parser, (void *) _xml_externalEntityRefHandler); @@ -1402,13 +1346,15 @@ PHP_FUNCTION(xml_set_external_entity_ref_handler) PHP_FUNCTION(xml_set_start_namespace_decl_handler) { xml_parser *parser; - zval *pind, **hdl; + zval *pind, *hdl; - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "rZ", &pind, &hdl) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "rz", &pind, &hdl) == FAILURE) { return; } - ZEND_FETCH_RESOURCE(parser,xml_parser *, &pind, -1, "XML Parser", le_xml_parser); + if ((parser = (xml_parser *)zend_fetch_resource(Z_RES_P(pind), "XML Parser", le_xml_parser)) == NULL) { + RETURN_FALSE; + } xml_set_handler(&parser->startNamespaceDeclHandler, hdl); XML_SetStartNamespaceDeclHandler(parser->parser, _xml_startNamespaceDeclHandler); @@ -1421,13 +1367,15 @@ PHP_FUNCTION(xml_set_start_namespace_decl_handler) PHP_FUNCTION(xml_set_end_namespace_decl_handler) { xml_parser *parser; - zval *pind, **hdl; + zval *pind, *hdl; - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "rZ", &pind, &hdl) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "rz", &pind, &hdl) == FAILURE) { return; } - ZEND_FETCH_RESOURCE(parser,xml_parser *, &pind, -1, "XML Parser", le_xml_parser); + if ((parser = (xml_parser *)zend_fetch_resource(Z_RES_P(pind), "XML Parser", le_xml_parser)) == NULL) { + RETURN_FALSE; + } xml_set_handler(&parser->endNamespaceDeclHandler, hdl); XML_SetEndNamespaceDeclHandler(parser->parser, _xml_endNamespaceDeclHandler); @@ -1442,16 +1390,20 @@ PHP_FUNCTION(xml_parse) xml_parser *parser; zval *pind; char *data; - int data_len, ret; - long isFinal = 0; + size_t data_len; + int ret; + zend_long isFinal = 0; - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "rs|l", &pind, &data, &data_len, &isFinal) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "rs|l", &pind, &data, &data_len, &isFinal) == FAILURE) { return; } - ZEND_FETCH_RESOURCE(parser,xml_parser *, &pind, -1, "XML Parser", le_xml_parser); + + if ((parser = (xml_parser *)zend_fetch_resource(Z_RES_P(pind), "XML Parser", le_xml_parser)) == NULL) { + RETURN_FALSE; + } parser->isparsing = 1; - ret = XML_Parse(parser->parser, data, data_len, isFinal); + ret = XML_Parse(parser->parser, (XML_Char*)data, data_len, isFinal); parser->isparsing = 0; RETVAL_LONG(ret); } @@ -1464,28 +1416,31 @@ PHP_FUNCTION(xml_parse) PHP_FUNCTION(xml_parse_into_struct) { xml_parser *parser; - zval *pind, **xdata, **info = NULL; + zval *pind, *xdata, *info = NULL; char *data; - int data_len, ret; + size_t data_len; + int ret; - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "rsZ|Z", &pind, &data, &data_len, &xdata, &info) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "rsz/|z/", &pind, &data, &data_len, &xdata, &info) == FAILURE) { return; } - + if (info) { - zval_dtor(*info); - array_init(*info); + zval_ptr_dtor(info); + array_init(info); } - ZEND_FETCH_RESOURCE(parser,xml_parser *, &pind, -1, "XML Parser", le_xml_parser); + if ((parser = (xml_parser *)zend_fetch_resource(Z_RES_P(pind), "XML Parser", le_xml_parser)) == NULL) { + RETURN_FALSE; + } - zval_dtor(*xdata); - array_init(*xdata); + zval_ptr_dtor(xdata); + array_init(xdata); - parser->data = *xdata; + ZVAL_COPY_VALUE(&parser->data, xdata); if (info) { - parser->info = *info; + ZVAL_COPY_VALUE(&parser->info, info); } parser->level = 0; @@ -1496,7 +1451,7 @@ PHP_FUNCTION(xml_parse_into_struct) XML_SetCharacterDataHandler(parser->parser, _xml_characterDataHandler); parser->isparsing = 1; - ret = XML_Parse(parser->parser, data, data_len, 1); + ret = XML_Parse(parser->parser, (XML_Char*)data, data_len, 1); parser->isparsing = 0; RETVAL_LONG(ret); @@ -1510,12 +1465,15 @@ PHP_FUNCTION(xml_get_error_code) xml_parser *parser; zval *pind; - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "r", &pind) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "r", &pind) == FAILURE) { return; } - ZEND_FETCH_RESOURCE(parser,xml_parser *, &pind, -1, "XML Parser", le_xml_parser); - RETVAL_LONG((long)XML_GetErrorCode(parser->parser)); + if ((parser = (xml_parser *)zend_fetch_resource(Z_RES_P(pind), "XML Parser", le_xml_parser)) == NULL) { + RETURN_FALSE; + } + + RETURN_LONG((zend_long)XML_GetErrorCode(parser->parser)); } /* }}} */ @@ -1523,16 +1481,16 @@ PHP_FUNCTION(xml_get_error_code) Get XML parser error string */ PHP_FUNCTION(xml_error_string) { - long code; + zend_long code; char *str; - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "l", &code) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "l", &code) == FAILURE) { return; } str = (char *)XML_ErrorString((int)code); if (str) { - RETVAL_STRING(str, 1); + RETVAL_STRING(str); } } /* }}} */ @@ -1544,10 +1502,13 @@ PHP_FUNCTION(xml_get_current_line_number) xml_parser *parser; zval *pind; - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "r", &pind) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "r", &pind) == FAILURE) { return; } - ZEND_FETCH_RESOURCE(parser,xml_parser *, &pind, -1, "XML Parser", le_xml_parser); + + if ((parser = (xml_parser *)zend_fetch_resource(Z_RES_P(pind), "XML Parser", le_xml_parser)) == NULL) { + RETURN_FALSE; + } RETVAL_LONG(XML_GetCurrentLineNumber(parser->parser)); } @@ -1560,10 +1521,13 @@ PHP_FUNCTION(xml_get_current_column_number) xml_parser *parser; zval *pind; - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "r", &pind) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "r", &pind) == FAILURE) { return; } - ZEND_FETCH_RESOURCE(parser,xml_parser *, &pind, -1, "XML Parser", le_xml_parser); + + if ((parser = (xml_parser *)zend_fetch_resource(Z_RES_P(pind), "XML Parser", le_xml_parser)) == NULL) { + RETURN_FALSE; + } RETVAL_LONG(XML_GetCurrentColumnNumber(parser->parser)); } @@ -1576,10 +1540,13 @@ PHP_FUNCTION(xml_get_current_byte_index) xml_parser *parser; zval *pind; - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "r", &pind) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "r", &pind) == FAILURE) { return; } - ZEND_FETCH_RESOURCE(parser,xml_parser *, &pind, -1, "XML Parser", le_xml_parser); + + if ((parser = (xml_parser *)zend_fetch_resource(Z_RES_P(pind), "XML Parser", le_xml_parser)) == NULL) { + RETURN_FALSE; + } RETVAL_LONG(XML_GetCurrentByteIndex(parser->parser)); } @@ -1591,23 +1558,25 @@ PHP_FUNCTION(xml_parser_free) { zval *pind; xml_parser *parser; + zend_resource *res; - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "r", &pind) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "r", &pind) == FAILURE) { return; } - ZEND_FETCH_RESOURCE(parser,xml_parser *, &pind, -1, "XML Parser", le_xml_parser); - - if (parser->isparsing == 1) { - php_error_docref(NULL TSRMLS_CC, E_WARNING, "Parser cannot be freed while it is parsing."); + if ((parser = (xml_parser *)zend_fetch_resource(Z_RES_P(pind), "XML Parser", le_xml_parser)) == NULL) { RETURN_FALSE; } - if (zend_list_delete(parser->index) == FAILURE) { + if (parser->isparsing == 1) { + php_error_docref(NULL, E_WARNING, "Parser cannot be freed while it is parsing."); RETURN_FALSE; } - RETVAL_TRUE; + res = Z_RES(parser->index); + ZVAL_UNDEF(&parser->index); + zend_list_close(res); + RETURN_TRUE; } /* }}} */ @@ -1616,40 +1585,43 @@ PHP_FUNCTION(xml_parser_free) PHP_FUNCTION(xml_parser_set_option) { xml_parser *parser; - zval *pind, **val; - long opt; + zval *pind, *val; + zend_long opt; - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "rlZ", &pind, &opt, &val) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "rlz", &pind, &opt, &val) == FAILURE) { return; } - ZEND_FETCH_RESOURCE(parser,xml_parser *, &pind, -1, "XML Parser", le_xml_parser); + + if ((parser = (xml_parser *)zend_fetch_resource(Z_RES_P(pind), "XML Parser", le_xml_parser)) == NULL) { + RETURN_FALSE; + } switch (opt) { case PHP_XML_OPTION_CASE_FOLDING: convert_to_long_ex(val); - parser->case_folding = Z_LVAL_PP(val); + parser->case_folding = Z_LVAL_P(val); break; case PHP_XML_OPTION_SKIP_TAGSTART: convert_to_long_ex(val); - parser->toffset = Z_LVAL_PP(val); + parser->toffset = Z_LVAL_P(val); break; case PHP_XML_OPTION_SKIP_WHITE: convert_to_long_ex(val); - parser->skipwhite = Z_LVAL_PP(val); + parser->skipwhite = Z_LVAL_P(val); break; case PHP_XML_OPTION_TARGET_ENCODING: { xml_encoding *enc; convert_to_string_ex(val); - enc = xml_get_encoding(Z_STRVAL_PP(val)); + enc = xml_get_encoding((XML_Char*)Z_STRVAL_P(val)); if (enc == NULL) { - php_error_docref(NULL TSRMLS_CC, E_WARNING, "Unsupported target encoding \"%s\"", Z_STRVAL_PP(val)); + php_error_docref(NULL, E_WARNING, "Unsupported target encoding \"%s\"", Z_STRVAL_P(val)); RETURN_FALSE; } parser->target_encoding = enc->name; break; } default: - php_error_docref(NULL TSRMLS_CC, E_WARNING, "Unknown option"); + php_error_docref(NULL, E_WARNING, "Unknown option"); RETURN_FALSE; break; } @@ -1663,22 +1635,25 @@ PHP_FUNCTION(xml_parser_get_option) { xml_parser *parser; zval *pind; - long opt; + zend_long opt; - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "rl", &pind, &opt) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "rl", &pind, &opt) == FAILURE) { return; } - ZEND_FETCH_RESOURCE(parser,xml_parser *, &pind, -1, "XML Parser", le_xml_parser); + + if ((parser = (xml_parser *)zend_fetch_resource(Z_RES_P(pind), "XML Parser", le_xml_parser)) == NULL) { + RETURN_FALSE; + } switch (opt) { case PHP_XML_OPTION_CASE_FOLDING: RETURN_LONG(parser->case_folding); break; case PHP_XML_OPTION_TARGET_ENCODING: - RETURN_STRING(parser->target_encoding, 1); + RETURN_STRING((char *)parser->target_encoding); break; default: - php_error_docref(NULL TSRMLS_CC, E_WARNING, "Unknown option"); + php_error_docref(NULL, E_WARNING, "Unknown option"); RETURN_FALSE; break; } @@ -1692,18 +1667,18 @@ PHP_FUNCTION(xml_parser_get_option) PHP_FUNCTION(utf8_encode) { char *arg; - XML_Char *encoded; - int arg_len, len; + size_t arg_len; + zend_string *encoded; - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s", &arg, &arg_len) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "s", &arg, &arg_len) == FAILURE) { return; } - encoded = xml_utf8_encode(arg, arg_len, &len, "ISO-8859-1"); + encoded = xml_utf8_encode(arg, arg_len, (XML_Char*)"ISO-8859-1"); if (encoded == NULL) { RETURN_FALSE; } - RETVAL_STRINGL(encoded, len, 0); + RETURN_STR(encoded); } /* }}} */ @@ -1712,18 +1687,18 @@ PHP_FUNCTION(utf8_encode) PHP_FUNCTION(utf8_decode) { char *arg; - XML_Char *decoded; - int arg_len, len; + size_t arg_len; + zend_string *decoded; - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "s", &arg, &arg_len) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "s", &arg, &arg_len) == FAILURE) { return; } - decoded = xml_utf8_decode(arg, arg_len, &len, "ISO-8859-1"); + decoded = xml_utf8_decode((XML_Char*)arg, arg_len, (XML_Char*)"ISO-8859-1"); if (decoded == NULL) { RETURN_FALSE; } - RETVAL_STRINGL(decoded, len, 0); + RETURN_STR(decoded); } /* }}} */ diff --git a/ext/xml/xml.mak b/ext/xml/xml.mak index ab60f28eed..b712dea1d6 100644 --- a/ext/xml/xml.mak +++ b/ext/xml/xml.mak @@ -4,7 +4,7 @@ PROJECT_ROOT = ..\.. # Module details MODULE_NAME = php_xml -MODULE_DESC = "PHP 5 - XML Extension" +MODULE_DESC = "PHP 7 - XML Extension" VMAJ = 3 VMIN = 0 VREV = 0 |