yeptris 0.6.15.1-aarch64-linux → 0.6.15.2-aarch64-linux

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: df5a484e47dccdf0bd9d05ff80c01220a75c5c1f34ceb453573c0be635009570
-  data.tar.gz: 726862488f90949222bce7c0a7c4ab1da885cfe872e630ed70ebf899e2503fe6
+  metadata.gz: 419470a0e038bec65880a3d764da46f072d7993e5aca9de71825a8428d5b2f98
+  data.tar.gz: 19cf714c7f80abc07d8729420700ef2c08bf1a140de03c3e11a8bb6ff177a4f4
 SHA512:
-  metadata.gz: 758dafc8487908e663705fb7a67fe08d2a825edb2e0a359163f5d591a1ddcb6d8d59c67f6f745ccc2dcdfc777bd3185419bb316c26d33df152a31834d80e536c
-  data.tar.gz: 76772fdb111e3fbbd6ccd556e8fa966bb7311010c729c35d996f1399eb5e4d64a70385336c2ca8cd3d71743fbac5ca50d7e8e06fd179396d9dd29115a3c363b9
+  metadata.gz: 9d11c53e5305c76f780f431d73be9c62228e250ab164c4fd636429ad4fca4e3705b1d46b0e9cf7abed5d8a0662d0347a2399de0ce3a16f5eec987d05d0fb053f
+  data.tar.gz: 5c24e95902b4ae44595710274ede3efbd605885d8cbc0747fc0965fcb8a2addb32ea8f18648606fef823d09a6f106d777d175f4e8ed0f1c8e74810f22d6cbee6

data/lib/yeptris/document.rb

@@ -20,7 +20,7 @@ class Yeptris::Document
     # always yields the same Ruby object (identity for aliases/eql?),
     # cleared at free — no stale entries, no GC-race weak maps.
     @wrapper_cache = {}
-    ObjectSpace.define_finalizer(self, self.class.finalize(c_ptr, freed)) unless c_ptr.null?
+    ObjectSpace.define_finalizer(self, self.class.finalize(c_ptr)) unless c_ptr.null?
   end
 
   def self.parse(yaml, schema: :core_12, max_depth: 0)
@@ -88,6 +88,9 @@ class Yeptris::Document
     return if @freed.state == :freed
 
     @freed.state = :freed
+    # stop the GC finalizer BEFORE the C free — it captures the same
+    # pointer, and its firing after this free is the double free
+    ObjectSpace.undefine_finalizer(self)
     @wrapper_cache.clear
     Yeptris::FFI.yeptris_document_free(@c_ptr)
   end
@@ -221,12 +224,15 @@ class Yeptris::Document
     self
   end
 
-  # GC safety net: an explicit #free already ran is fine; a miss here
-  # frees C memory that would otherwise leak.
-  def self.finalize(c_ptr, freed)
-    proc do
-      Yeptris::FFI.yeptris_document_free(c_ptr) if freed.state == :alive
-      freed.state = :freed
-    end
+  # GC safety net (#182): the proc captures ONLY the raw pointer — no
+  # Ruby objects, no wrapper, no freed-flag struct. A finalizer that
+  # closes over the owning wrapper races that wrapper's own sweep
+  # (the classic use-after-free: freed?/free against half-collected
+  # ivars, then a garbage C pointer into yeptris_document_free →
+  # SIGABRT). Explicit #free undefines this finalizer BEFORE the C
+  # free, so the double free cannot happen; finalizers run on the
+  # main thread, so there is no cross-thread window.
+  def self.finalize(c_ptr)
+    proc { Yeptris::FFI.yeptris_document_free(c_ptr) }
   end
 end

data/lib/yeptris/materializer.rb

@@ -115,7 +115,11 @@ module Yeptris
       end
 
       def parse_timestamp(v)
-        return Date.parse(v) unless v.match?(/[Tt ]\d/)
+        # stdlib scalar_scanner: date-only strings parse with
+        # strptime('%F', Date::GREGORIAN) — the PROLEPTIC calendar —
+        # not Date.parse's default ITALY reform (a 1582 cycle fails
+        # the ported test_julian_date otherwise)
+        return ::Date.strptime(v, "%F", ::Date::GREGORIAN) unless v.match?(/[Tt ]\d/)
 
         # normalize the YAML 1.1 space forms onto iso8601 for
         # xmlschema: "2001-12-14 21:59:43.10 -05:00" ->

data/lib/yeptris/node.rb

@@ -394,6 +394,15 @@ class Yeptris::Node
   def scalar_to_ruby
     return symbolize if symbol? && tag_id == :str
 
+    # stdlib deserialize's tagged-scalar arms: the class tag carries
+    # DateTime (dumped tagged — a plain timestamp re-loads as Time)
+    if tag == "!ruby/object:DateTime"
+      ts = value.to_s.sub(/ (\d)/, 'T\\1').sub(/ ([+-]\d)/, '\\1')
+      t = ::Time.xmlschema(ts)
+      return ::DateTime.civil(t.year, t.month, t.day, t.hour, t.min, t.sec,
+                              Rational(t.utc_offset, 86_400)) + Rational(t.subsec, 86_400)
+    end
+
     case tag_id
     when :null then nil
     when :bool then to_bool

data/lib/yeptris/psych/visitors.rb

@@ -66,11 +66,34 @@ module Yeptris
           end
         end
 
+        # stdlib register()s every object — scalars included — so a
+        # repeated scalar dumps &name/*name (test_float_references,
+        # test_alias_with_time). Containers already route through
+        # anchor_for; this is the scalar arm of the same machinery.
+        def anchored_scalar(obj, text: nil, tag: nil)
+          state, name = anchor_for(obj)
+          return alias_of(obj, name) if state == :alias
+
+          n = scalar(text || obj, tag: tag)
+          n.set_anchor(name) if name
+          remember(obj, n)
+          n
+        end
+
         def visit(obj)
           case obj
+          # strings/floats/ints: stdlib yaml_tree does NOT anchor them
+          # (its float test is load-side only) — and CBOR encode
+          # rejects the anchors the cbor profile feeds it
           when nil, true, false, ::Integer, ::Float, ::String then scalar(obj)
           when ::Symbol then @tree.new_scalar(":#{obj}", :plain) # psych emits symbols bare, never through visit_String's quoting rules
-          when ::Date, ::Time then scalar(obj) # timestamp text, never ivars
+          # stdlib visit_DateTime: the class tag carries the type (a
+          # plain timestamp re-loads as Time) — "!ruby/object:DateTime
+          # 2017-04-13 12:00:00.500000000 +09:00", to_s text, one line
+          when ::DateTime
+            anchored_scalar(obj, text: obj.strftime("%F %H:%M:%S.%9N %:z"),
+                            tag: "!ruby/object:DateTime")
+          when ::Date, ::Time then anchored_scalar(obj) # timestamp text, never ivars
           when ::Hash then visit_hash(obj)
           when ::Array then visit_array(obj)
           when ::Struct then visit_struct(obj)
@@ -101,7 +124,7 @@ module Yeptris
           if @refs[oid] < 2
             return [:none, nil]
           end
-          name = "o#{@counter += 1}"
+          name = (@counter += 1).to_s # stdlib: &1, *1
           @names[oid] = name
           [:new, name]
         end
@@ -156,7 +179,7 @@ module Yeptris
               # neutral surface alone was pinned)
               ::Yeptris::YAML::BulkBuilder.time_text(obj)
             elsif obj.is_a?(::Date)
-              obj.iso8601 # canonical calendar form (DateTime rides here too)
+              obj.iso8601 # canonical calendar form
             elsif obj.nil?
               "" # libyaml's null rendering rides bare (issue #290)
             elsif obj.is_a?(::Float)
@@ -165,7 +188,7 @@ module Yeptris
               obj.to_s
             end
           n =
-            if obj.is_a?(::String)
+            if obj.is_a?(::String) && !tag
               ::Yeptris::YAML::Builder.build_string(@tree, obj)
             else
               @tree.new_scalar(text, :plain)
@@ -220,7 +243,7 @@ module Yeptris
           m = @tree.new_mapping
           remember(strct, m)
           m.set_anchor(name) if name
-          m.set_tag(strct.class.name ? "!ruby/struct:#{strct.class.name}" : "!ruby/struct")
+          m.set_tag(strct.class.name.to_s.empty? ? "!ruby/struct" : "!ruby/struct:#{strct.class.name}")
           strct.each_pair { |k, v| m.map_add(key_text(k), visit(v)) }
           m
         end
@@ -326,6 +349,9 @@ module Yeptris
             when ::Yeptris::Psych::Nodes::Mapping then visit_mapping(node)
             else node&.to_ruby
             end
+          # stdlib register(): an anchored node's result is THE object
+          # for every alias to that anchor (scalars included)
+          anchors[node.anchor] = result if node.respond_to?(:anchor) && node.anchor
           # the domain-types post-pass (stdlib ToRuby#accept's tail):
           # registered blocks transform the revived result per node
           if node && !::Yeptris::Psych.domain_types.empty?

data/lib/yeptris/psych.rb

@@ -299,10 +299,11 @@ module Yeptris
         (0...doc.document_count).each do |i|
           stream.children << Nodes::Builder.document_stream_child(doc, i)
         end
+        # ownership: the DOCUMENT wrapper is the sole owner — its own
+        # finalizer (pointer-only closure) frees the C memory; the
+        # stream merely references it. A tree-side finalizer closing
+        # over the wrapper raced the wrapper's sweep (#182's abort).
         stream.owner = doc
-        ObjectSpace.define_finalizer(
-          stream, proc { doc.free unless doc.freed? }
-        )
         stream
       rescue Yeptris::ParseError => e
         raise SyntaxError.from_parse_error(e)
@@ -327,6 +328,11 @@ module Yeptris
         # data anyway
         out =
           case obj
+          # DateTime < Date in ruby's hierarchy — the fast scalar
+          # path would emit it tag-less iso8601 (re-loading as Time);
+          # the visitor carries stdlib's !ruby/object:DateTime tag
+          when ::DateTime
+            Visitors::YAMLTree.new.push(obj).finish
           when nil, true, false, ::String, ::Integer, ::Float, ::Symbol, ::Date, ::Time
             Yeptris::YAML.dump(obj, header: true)
           else
@@ -442,12 +448,13 @@ module Yeptris
           @tags = tags
         end
 
-        # @api private — transfer of C ownership to this tree
+        # @api private — attach the document: the tree REFERENCES it
+        # (keeping the C memory alive while the tree is reachable);
+        # the wrapper's own finalizer is the sole freer (#182: a
+        # tree-side finalizer closing over the wrapper raced the
+        # wrapper's sweep — SIGABRT under document churn)
         def own(yeptris_doc)
           @owner = yeptris_doc
-          ObjectSpace.define_finalizer(
-            self, proc { yeptris_doc.free unless yeptris_doc.freed? }
-          )
           self
         end
 

data/lib/yeptris.rb

@@ -3,7 +3,7 @@
 module Yeptris
   # The gem's version lives in the parent namespace's file — the last
   # internal require (yeptris/version) retired with it.
-  VERSION = "0.6.15.1".freeze
+  VERSION = "0.6.15.2".freeze
   # The error hierarchy lives in THIS file (the parent namespace's
   # own file): nested constants do not trigger a parent-constant
   # autoload, and the law forbids internal requires — defining the

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: yeptris
 version: !ruby/object:Gem::Version
-  version: 0.6.15.1
+  version: 0.6.15.2
 platform: aarch64-linux
 authors:
 - Ribose Inc.