leptris 1.9.365.1 → 1.9.366.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 973845563957861b68366fd3c90f3a8d3f1a5ba5921e09f2a2dc268542faa8b6
-  data.tar.gz: 71814d603d2c40586dc2e3c3aa9d36750e67a8cdb54864257a9f56cfa076a6c8
+  metadata.gz: 97ab0ffc1ab21b11d43593a97f8d91f75f49028460d68d5f0fdf7dbe14f697f7
+  data.tar.gz: 3badee92d7d7744c93da9434badb3401e7b4e5b4f61acc3e984029ed16dcf903
 SHA512:
-  metadata.gz: f298b9d5e3ac0ffd299ee2323b4efcadddd2c7c7f09be765009c3764d99bbfeb88e3720a92888fbfda3195dfa5bededeca05e38c1790c7c2071fd87807718ed8
-  data.tar.gz: ea9086febdac6a55cb745ce9c240ed7ea2db9ac26843b6b1bfcca43c873a9fbcce1f522dff609ef83a8c62e9af98120d6e9dad18649ba0ec3ea77556c215d0cd
+  metadata.gz: 9222fe57cebd087fe04c892116bd771266c1671039d2c8a6bf7fd0846b746210766adc8db5c7824289e9cd7e48cb17effb3ec37a2505732f29da52068ccf7fe9
+  data.tar.gz: 6f9fc5fc3e204d9112e9f49bdd829b3a0c11360135a8037f471753c01e414e688a6b94ee35f2bf89113898d0f702119f1f076e4716dac17fa2da9077bbbdd4b5

data/Rakefile

@@ -8,7 +8,7 @@ RSpec::Core::RakeTask.new(:spec)
 # Pin for `rake compile` and the platform-gem builds. Keep in lockstep
 # with .github/workflows/build.yml (which calls `rake compile`) and the
 # CHANGELOG when libleptris releases.
-LIBLEPTRIS_VERSION = "1.9.365"
+LIBLEPTRIS_VERSION = "1.9.366"
 # Vendored alongside libleptris for fn:normalize-unicode (TODO
 # .restructure/20): built per platform with a RELOCATABLE @rpath
 # install name, loaded by ffi.rb before libleptris so the

data/lib/leptris/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Leptris
-  VERSION = "1.9.365.1"
+  VERSION = "1.9.366.1"
 end

data/vendor-src/README.md

@@ -1,7 +1,7 @@
 Sources vendored in this gem (recompile rights; the ruby
 variant builds them at install via extconf.rb):
 
-- libleptris v1.9.365
+- libleptris v1.9.366
 - utf8proc v2.11.0
 
 Rebuild by hand:

data/vendor-src/libleptris/CHANGELOG.md

@@ -1,5 +1,27 @@
 ## [Unreleased]
 
+## [1.9.366] - 2026-10-17
+
+### Fixed
+
+- **dom: the foreign leaf adopt copies exactly the argument node (#1755, PR #1757).**
+  `adopt_for_splice`'s non-element branch ran a wholesale sibling-chain copy
+  (`copy_leaf_chain`) whose switch had no ELEMENT case: any mixed-content
+  chain — a text node whose next sibling is the element that follows it, the
+  standard fragment shape lutaml/moxml produce — hit `default:` and the whole
+  append failed with `LEPTRIS_ERROR_MEMORY` ("Memory allocation failed": a
+  lie; nothing was out of memory). The run-copy was also wrong for every
+  caller that reaches this path — the binding's fragment loop and moxml's
+  NodeSet iteration append fragment children individually, so a chain copy
+  duplicated every later sibling even where it succeeded. The foreign leaf
+  adopt now copies exactly the argument (doc-level leaves were already
+  single-node for the same reason, #1539); the single-caller chain copier is
+  deleted. This is the deterministic face of the Ruby 4.0 render crash
+  (leptris-ruby#489): the GC-stressed metanorma collection render goes
+  2 failures -> 0. Engine suite 2048/2048; binding suite 1094/1094.
+
+
+
 ## [1.9.365] - 2026-10-16
 
 ### Fixed

data/vendor-src/libleptris/CMakeLists.txt

@@ -3,7 +3,7 @@
 cmake_minimum_required(VERSION 3.20)
 
 project(leptris
-    VERSION 1.9.365
+    VERSION 1.9.366
     DESCRIPTION "Fast XML parser and XPath evaluator in pure C"
     LANGUAGES C CXX
 )

data/vendor-src/libleptris/src/leptris/dom/element_modify.c

@@ -359,74 +359,6 @@ static LeptrisNode* copy_leaf_one(LeptrisNode* c,
     }
 }
 
-/* #1534: copy a non-element node (plus its linked tail — the #540
- * chain shape) into the target document's pool. Same per-kind
- * creation helpers the subtree copier uses. */
-static LeptrisNode* copy_leaf_chain(LeptrisNode* head,
-                                    struct leptris_document* target) {
-    LeptrisNode* copy_head = NULL;
-    LeptrisNode* copy_tail = NULL;
-    for (LeptrisNode* c = head; c;
-         c = leptris_node_get_next_sibling(c)) {
-        LeptrisNode* cc = NULL;
-        switch (c->type) {
-            case LEPTRIS_NODE_TYPE_TEXT: {
-                LeptrisTextNode* t = (LeptrisTextNode*)c;
-                const char* content = leptris_textnode_content(t);
-                cc = (LeptrisNode*)leptris_text_create(
-                    content, t->content_len, target->pool, target);
-                break;
-            }
-            case LEPTRIS_NODE_TYPE_CDATA: {
-                LeptrisCDATANode* cd = (LeptrisCDATANode*)c;
-                cc = (LeptrisNode*)leptris_cdata_create(
-                    cd->content, cd->content ? strlen(cd->content) : 0,
-                    target->pool);
-                break;
-            }
-            case LEPTRIS_NODE_TYPE_COMMENT: {
-                LeptrisCommentNode* cm = (LeptrisCommentNode*)c;
-                cc = (LeptrisNode*)leptris_comment_create(
-                    cm->content,
-                    cm->content ? strlen(cm->content) : 0,
-                    target->pool);
-                break;
-            }
-            case LEPTRIS_NODE_TYPE_PI: {
-                LeptrisPINode* pi = (LeptrisPINode*)c;
-                cc = (LeptrisNode*)leptris_pi_create(
-                    pi->target, pi->target ? strlen(pi->target) : 0,
-                    pi->data ? pi->data : "",
-                    pi->data ? strlen(pi->data) : 0, target->pool);
-                break;
-            }
-            case LEPTRIS_NODE_TYPE_ENTITY_REF: {
-                LeptrisEntityRefNode* er = (LeptrisEntityRefNode*)c;
-                cc = (LeptrisNode*)leptris_entity_ref_create(
-                    er->name, er->name ? strlen(er->name) : 0,
-                    target->pool);
-                break;
-            }
-            default:
-                cc = NULL;
-        }
-        if (!cc) {
-            /* OOM mid-chain: copies already made live only in the
-             * target pool — no caller-visible handle exists, and
-             * document free reclaims them (the same conservatism
-             * copy_subtree_detached uses on !cc). */
-            return NULL;
-        }
-        if (!copy_head) {
-            copy_head = cc;
-        } else {
-            leptris_node_set_next_sibling(copy_tail, cc);
-        }
-        copy_tail = cc;
-    }
-    return copy_head;
-}
-
 static LeptrisNode* adopt_for_splice(LeptrisNode* node,
                                      struct leptris_document* target) {
     if (!node || !target) return node;
@@ -465,13 +397,15 @@ static LeptrisNode* adopt_for_splice(LeptrisNode* node,
          * the target — its pool dies with the target, so the node
          * moves by reference. */
         if (src->absorbed_by == target) return node;
-        /* top == node: a DOC-LEVEL leaf (prolog/epilog, #580) —
-         * its sibling chain spans unrelated nodes (including the
-         * root element), so copy exactly the argument (#1539).
-         * An in-element leaf keeps the #540 chain shape. */
-        if (top == node) return copy_leaf_one(node, target);
-        LeptrisNode* copy = copy_leaf_chain(node, target);
-        return copy;
+        /* Copy exactly the argument (#1755): the callers of this
+         * face append fragment children INDIVIDUALLY (the binding's
+         * fragment loop, moxml's NodeSet iteration), so a wholesale
+         * run copy duplicated every later sibling — and its missing
+         * ELEMENT case failed the whole append with
+         * LEPTRIS_ERROR_MEMORY on the first mixed-content chain.
+         * Doc-level leaves (prolog/epilog, #580) were already
+         * single-node for the same reason (#1539). */
+        return copy_leaf_one(node, target);
     }
     /* Type-safe walk to the chain top: get_document reads
      * element-form fields and misreads non-element structs. */

data/vendor-src/libleptris/test/dom/test_dom.cpp

@@ -3019,6 +3019,58 @@ TEST(DomAttachInstalled, ForeignAttachReportsCopyAddress) {
     leptris_document_free(target);
 }
 
+/* leptris#1755: a foreign MIXED-CONTENT fragment hands the
+ * engine a text node whose sibling chain carries the following
+ * element. The leaf adopt ran a WHOLESALE chain copy that had no
+ * ELEMENT case — the element hit default: and the whole append
+ * failed with LEPTRIS_ERROR_MEMORY ("Memory allocation failed" —
+ * a lie; nothing was out of memory). Every caller of this face
+ * appends fragment children INDIVIDUALLY (the binding's fragment
+ * loop, moxml's NodeSet iteration), so the foreign leaf adopt
+ * copies exactly the argument node — the sibling run stays in the
+ * source for its own appends. */
+TEST(DomAttachInstalled, ForeignMixedChainTextBeforeElement) {
+    LeptrisStatus st = LEPTRIS_OK;
+    const char* src = "<f>lead<item>t</item>tail</f>";
+    LeptrisDocument frag = leptris_parse_string(src, strlen(src), &st);
+    ASSERT_NE(frag, nullptr);
+    LeptrisDocument target = leptris_parse_string("<doc/>", 6, &st);
+    ASSERT_NE(target, nullptr);
+    LeptrisElement troot = leptris_document_root(target);
+    LeptrisElement froot = leptris_document_root(frag);
+    /* node-level first child: the TEXT "lead" (the element-level
+     * face would skip it and return <item>). */
+    LeptrisNodeRef text = leptris_node_first_child((LeptrisNodeRef)froot);
+    ASSERT_NE(text, nullptr);
+    ASSERT_EQ(leptris_node_get_type(text), LEPTRIS_NODE_TYPE_TEXT);
+
+    EXPECT_EQ(leptris_element_add_child_addr(
+                  (uintptr_t)troot, (uintptr_t)text),
+              LEPTRIS_OK);
+    /* exactly the argument: no sibling run rides along (serialize
+     * pins the child set; child_count is element-oriented) */
+    char* out = leptris_element_serialize(troot, nullptr);
+    ASSERT_NE(out, nullptr);
+    EXPECT_STREQ(out, "<doc>lead</doc>");
+    leptris_free_string(out);
+
+    /* the binding's loop continues: the element sibling appends on
+     * its own, and the trailing text with it — no duplication of
+     * the element the first adopt must NOT have carried */
+    LeptrisElement item = leptris_element_first_child_any(froot);
+    ASSERT_NE(item, nullptr);
+    EXPECT_EQ(leptris_element_add_child_addr(
+                  (uintptr_t)troot, (uintptr_t)item),
+              LEPTRIS_OK);
+    /* the source dies; the installed copies must survive */
+    leptris_document_free(frag);
+    out = leptris_element_serialize(troot, nullptr);
+    ASSERT_NE(out, nullptr);
+    EXPECT_STREQ(out, "<doc>lead<item>t</item></doc>");
+    leptris_free_string(out);
+    leptris_document_free(target);
+}
+
 TEST(DomAttachInstalled, SameDocumentReturnsArgument) {
     LeptrisStatus st = LEPTRIS_OK;
     LeptrisDocument d = leptris_parse_string("<r/>", 4, &st);

data/vendor-src/libleptris/vcpkg.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://raw.githubusercontent.com/microsoft/vcpkg/master/scripts/vcpkg.schema.json",
   "name": "leptris",
-  "version": "1.9.365",
+  "version": "1.9.366",
   "description": "High-performance XML parser and XPath 1.0 engine in C (libleptris)",
   "homepage": "https://github.com/leptris/leptris",
   "license": "MIT",

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: leptris
 version: !ruby/object:Gem::Version
-  version: 1.9.365.1
+  version: 1.9.366.1
 platform: ruby
 authors:
 - Ribose Inc.
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-10-16 00:00:00.000000000 Z
+date: 2026-10-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ffi