Merge branch 'PHP-8.2'

* PHP-8.2:
  Fix phpGH-11455: Segmentation fault with custom object date properties
  Revert "Fix phpGH-11404: DOMDocument::savexml and friends ommit xmlns="" declaration for null namespace, creating incorrect xml representation of the DOM"

Author: ndossche

ext/date/php_date.c

@@ -2333,7 +2333,9 @@ static void add_common_properties(HashTable *myht, zend_object *zobj)
 	common = zend_std_get_properties(zobj);
 
 	ZEND_HASH_MAP_FOREACH_STR_KEY_VAL_IND(common, name, prop) {
-		zend_hash_add(myht, name, prop);
+		if (zend_hash_add(myht, name, prop) != NULL) {
+			Z_TRY_ADDREF_P(prop);
+		}
 	} ZEND_HASH_FOREACH_END();
 }
 

ext/date/tests/gh11455.phpt

@@ -0,0 +1,39 @@
+--TEST--
+Bug GH-11455 (PHP 8.2 Segmentation fault on nesbot/carbon)
+--FILE--
+<?php
+class MyDateTimeImmutable extends DateTimeImmutable {
+    public function __construct(
+        string $datetime = "now",
+        ?DateTimeZone $timezone = null,
+        public ?stdClass $myProperty = null,
+    ) {
+        parent::__construct($datetime, $timezone);
+    }
+}
+
+$datetime = new MyDateTimeImmutable('2022-12-22T11:26:00Z', myProperty: new stdClass);
+$datetime->myProperty->field = str_repeat("hello", 3);
+$serialized = serialize($datetime);
+var_dump($datetime->myProperty);
+$unserialized = unserialize($serialized);
+var_dump($unserialized);
+?>
+--EXPECT--
+object(stdClass)#2 (1) {
+  ["field"]=>
+  string(15) "hellohellohello"
+}
+object(MyDateTimeImmutable)#3 (4) {
+  ["myProperty"]=>
+  object(stdClass)#4 (1) {
+    ["field"]=>
+    string(15) "hellohellohello"
+  }
+  ["date"]=>
+  string(26) "2022-12-22 11:26:00.000000"
+  ["timezone_type"]=>
+  int(2)
+  ["timezone"]=>
+  string(1) "Z"
+}

ext/dom/document.c

@@ -877,10 +877,6 @@ PHP_METHOD(DOMDocument, createElementNS)
 
 	if (errorcode == 0) {
 		if (xmlValidateName((xmlChar *) localname, 0) == 0) {
-			/* https://dom.spec.whatwg.org/#validate-and-extract: demands us to set an empty string uri to NULL */
-			if (uri_len == 0) {
-				uri = NULL;
-			}
 			nodep = xmlNewDocNode(docp, NULL, (xmlChar *) localname, (xmlChar *) value);
 			if (nodep != NULL && uri != NULL) {
 				xmlNsPtr nsptr = xmlSearchNsByHref(nodep->doc, nodep, (xmlChar *) uri);

ext/dom/element.c

@@ -56,10 +56,6 @@ PHP_METHOD(DOMElement, __construct)
 	if (uri_len > 0) {
 		errorcode = dom_check_qname(name, &localname, &prefix, uri_len, name_len);
 		if (errorcode == 0) {
-			/* https://dom.spec.whatwg.org/#validate-and-extract: demands us to set an empty string uri to NULL */
-			if (uri_len == 0) {
-				uri = NULL;
-			}
 			nodep = xmlNewNode (NULL, (xmlChar *)localname);
 			if (nodep != NULL && uri != NULL) {
 				nsptr = dom_get_ns(nodep, uri, &errorcode, prefix);

ext/dom/node.c

@@ -532,6 +532,7 @@ Since: DOM Level 2
 int dom_node_namespace_uri_read(dom_object *obj, zval *retval)
 {
 	xmlNode *nodep = dom_object_get_node(obj);
+	char *str = NULL;
 
 	if (nodep == NULL) {
 		php_dom_throw_error(INVALID_STATE_ERR, 1);
@@ -543,19 +544,20 @@ int dom_node_namespace_uri_read(dom_object *obj, zval *retval)
 		case XML_ATTRIBUTE_NODE:
 		case XML_NAMESPACE_DECL:
 			if (nodep->ns != NULL) {
-				char *str = (char *) nodep->ns->href;
-				/* https://dom.spec.whatwg.org/#concept-attribute: namespaceUri is "null or a non-empty string" */
-				if (str != NULL && str[0] != '\0') {
-					ZVAL_STRING(retval, str);
-					return SUCCESS;
-				}
+				str = (char *) nodep->ns->href;
 			}
 			break;
 		default:
+			str = NULL;
 			break;
 	}
 
-	ZVAL_NULL(retval);
+	if (str != NULL) {
+		ZVAL_STRING(retval, str);
+	} else {
+		ZVAL_NULL(retval);
+	}
+
 	return SUCCESS;
 }
 

ext/dom/php_dom.c

@@ -1497,33 +1497,13 @@ static void dom_libxml_reconcile_ensure_namespaces_are_declared(xmlNodePtr nodep
 	xmlDOMWrapReconcileNamespaces(&dummy_ctxt, nodep, /* options */ 0);
 }
 
-static bool dom_must_replace_namespace_by_empty_default(xmlDocPtr doc, xmlNodePtr nodep)
-{
-	xmlNsPtr default_ns = xmlSearchNs(doc, nodep->parent, NULL);
-	return default_ns != NULL && default_ns->href != NULL && default_ns->href[0] != '\0';
-}
-
-static void dom_replace_namespace_by_empty_default(xmlDocPtr doc, xmlNodePtr nodep)
-{
-	ZEND_ASSERT(nodep->ns == NULL);
-	/* The node uses the default empty namespace, but the current default namespace is non-empty.
-	 * We can't unconditionally do this because otherwise libxml2 creates an xmlns="" declaration.
-	 * Note: there's no point searching the oldNs list, because we haven't found it in the tree anyway.
-	 *       Ideally this would be pre-allocated but unfortunately libxml2 doesn't offer such a functionality. */
-	xmlSetNs(nodep, xmlNewNs(nodep, (const xmlChar *) "", NULL));
-}
-
 void dom_reconcile_ns(xmlDocPtr doc, xmlNodePtr nodep) /* {{{ */
 {
 	/* Although the node type will be checked by the libxml2 API,
 	 * we still want to do the internal reconciliation conditionally. */
 	if (nodep->type == XML_ELEMENT_NODE) {
 		dom_reconcile_ns_internal(doc, nodep, nodep->parent);
 		dom_libxml_reconcile_ensure_namespaces_are_declared(nodep);
-		/* Check nodep->ns first to avoid an expensive lookup. */
-		if (nodep->ns == NULL && dom_must_replace_namespace_by_empty_default(doc, nodep)) {
-			dom_replace_namespace_by_empty_default(doc, nodep);
-		}
 	}
 }
 /* }}} */
@@ -1547,30 +1527,12 @@ static void dom_reconcile_ns_list_internal(xmlDocPtr doc, xmlNodePtr nodep, xmlN
 
 void dom_reconcile_ns_list(xmlDocPtr doc, xmlNodePtr nodep, xmlNodePtr last)
 {
-	bool did_compute_must_replace_namespace_by_empty_default = false;
-	bool must_replace_namespace_by_empty_default = false;
-
 	dom_reconcile_ns_list_internal(doc, nodep, last, nodep->parent);
-
 	/* The loop is outside of the recursion in the above call because
 	 * dom_libxml_reconcile_ensure_namespaces_are_declared() performs its own recursion. */
 	while (true) {
 		/* The internal libxml2 call will already check the node type, no need for us to do it here. */
 		dom_libxml_reconcile_ensure_namespaces_are_declared(nodep);
-
-		/* We don't have to handle the children, because if their ns's are NULL they'll just take on the default
-		 * which should've been reconciled before. */
-		if (nodep->ns == NULL) {
-			/* This is an optimistic approach: we assume that most of the time we don't need the result of the computation. */
-			if (!did_compute_must_replace_namespace_by_empty_default) {
-				did_compute_must_replace_namespace_by_empty_default = true;
-				must_replace_namespace_by_empty_default = dom_must_replace_namespace_by_empty_default(doc, nodep);
-			}
-			if (must_replace_namespace_by_empty_default) {
-				dom_replace_namespace_by_empty_default(doc, nodep);
-			}
-		}
-
 		if (nodep == last) {
 			break;
 		}

ext/dom/tests/bug47530.phpt

@@ -121,7 +121,7 @@ test_appendChild_with_shadowing();
 <html xmlns="https://php.net/something" xmlns:ns="https://php.net/whatever"><element ns:foo="https://php.net/bar"/></html>
 -- Test document fragment without import --
 <?xml version="1.0"?>
-<html xmlns=""><element xmlns:foo="https://php.net/bar"><foo:bar/><bar/></element></html>
+<html xmlns=""><element xmlns:foo="https://php.net/bar"><foo:bar/><bar xmlns=""/></element></html>
 string(7) "foo:bar"
 string(19) "https://php.net/bar"
 -- Test document import --