ox 2.14.12 → 2.14.14

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bf4c5ff67edda493ebf9fd2a29fcee1584b0310f917094dec0a57eb93632f412
-  data.tar.gz: 7283aca3a2a2770f72696410a6691c131cfa2777562cd882f76bb0ea643c414e
+  metadata.gz: e05d933efc3ea58f72d9c9d9851d6dab608613f7614a09fa6b4dbed5b998b094
+  data.tar.gz: 9f85899f321b465fb0cb3b65e1a39d0182b9171505b04758bae30e24a4c1f76d
 SHA512:
-  metadata.gz: e6ada04ac77531e74f5d40995d660d6e993d9c69685c586d15cbd9c896dfe6c60df879a51adecf5f5d3cb860b8bcb3fd80116a1001ae6eb5d15fbeb472be7118
-  data.tar.gz: 4053266ff36240627decb9eefbdc30e1afcb07bf36fed7ad639d2e1cfbff5165991f59ee1fdc985259f6dc60e4394e4782df98de9c73c2e51be911f39436d595
+  metadata.gz: a19ec4e471fd35baba036fcdb01bc1d01820a0aac236a94d4da4cf10fde4da49a2c6cd144c1407ba42605b50c1239587bcee993aba5edb36219f65577742173b
+  data.tar.gz: dd1839eab2fef06690186839f81a17ed364c4713e9029960b81caf66ddc818b8892429eb60c1a0673de81ef53398946a5da89f2775078ad5fe85c4fbb2386b38

data/CHANGELOG.md

@@ -2,6 +2,18 @@
 
 All changes to the Ox gem are documented here. Releases follow semantic versioning.
 
+## [2.14.14] - 2023-01-26
+
+### Fixed
+
+- Change free to xfree on ruby alloced memory.
+
+## [2.14.13] - 2023-01-16
+
+### Fixed
+
+- Fixed the intern cache to handle symbol memory changes.
+
 ## [2.14.12] - 2022-12-27
 
 ### Fixed

data/ext/ox/hash_load.c

@@ -35,7 +35,7 @@ mark_value(PInfo pi, VALUE val) {
 	pi->mark_size = MARK_INC;
     } else if (pi->mark_size <= pi->mark_cnt) {
 	pi->mark_size += MARK_INC;
-	pi->marked = REALLOC_N(pi->marked, VALUE, pi->mark_size);
+	REALLOC_N(pi->marked, VALUE, pi->mark_size);
     }
     pi->marked[pi->mark_cnt] = val;
     pi->mark_cnt++;

data/ext/ox/sax.c

@@ -762,6 +762,7 @@ CB:
  */
 static char read_element_start(SaxDrive dr) {
     const char    *ename = 0;
+    size_t         nlen;
     volatile VALUE name  = Qnil;
     char           c;
     int            closed;
@@ -790,6 +791,7 @@ static char read_element_start(SaxDrive dr) {
         0 == strcasecmp("html", dr->buf.str)) {
         dr->options.hints = ox_hints_html();
     }
+    nlen = dr->buf.tail - dr->buf.str - 1;
     if (NULL != dr->options.hints) {
         hint_clear_empty(dr);
         h = ox_hint_find(dr->options.hints, dr->buf.str);
@@ -810,7 +812,7 @@ static char read_element_start(SaxDrive dr) {
                 if (rb_respond_to(dr->handler, ox_abort_id)) {
                     VALUE args[1];
 
-                    args[0] = str2sym(dr, dr->buf.str, dr->buf.tail - dr->buf.str - 1, NULL);
+                    args[0] = str2sym(dr, dr->buf.str, nlen, NULL);
                     rb_funcall2(dr->handler, ox_abort_id, 1, args);
                 }
                 dr->abort = true;
@@ -861,7 +863,7 @@ static char read_element_start(SaxDrive dr) {
             }
         }
     }
-    name = str2sym(dr, dr->buf.str, dr->buf.tail - dr->buf.str - 1, &ename);
+    name = str2sym(dr, dr->buf.str, nlen, &ename);
     if (dr->has_start_element && 0 >= dr->blocked &&
         (NULL == h || ActiveOverlay == h->overlay || NestOverlay == h->overlay)) {
         VALUE args[1];
@@ -894,7 +896,7 @@ static char read_element_start(SaxDrive dr) {
     } else if (stackless) {
         end_element_cb(dr, name, pos, line, col, h);
     } else if (NULL != h && h->jump) {
-        stack_push(&dr->stack, ename, name, h);
+        stack_push(&dr->stack, ename, nlen, name, h);
         if ('>' != c) {
             ox_sax_drive_error(dr, WRONG_CHAR "element not closed");
             return c;
@@ -902,7 +904,7 @@ static char read_element_start(SaxDrive dr) {
         read_jump(dr, h->name);
         return '<';
     } else {
-        stack_push(&dr->stack, ename, name, h);
+        stack_push(&dr->stack, ename, nlen, name, h);
     }
     if ('>' != c) {
         ox_sax_drive_error(dr, WRONG_CHAR "element not closed");

data/ext/ox/sax_stack.h

@@ -6,11 +6,16 @@
 #ifndef OX_SAX_STACK_H
 #define OX_SAX_STACK_H
 
+#include <stdlib.h>
+
+#include "intern.h"
 #include "sax_hint.h"
 
 #define STACK_INC	32
+#define NV_BUF_MAX      64
 
 typedef struct _nv {
+    char	name_buf[NV_BUF_MAX];
     const char	*name;
     VALUE	val;
     int		childCnt;
@@ -44,7 +49,7 @@ stack_cleanup(NStack stack) {
 }
 
 inline static void
-stack_push(NStack stack, const char *name, VALUE val, Hint hint) {
+stack_push(NStack stack, const char *name, size_t nlen, VALUE val, Hint hint) {
     if (stack->end <= stack->tail) {
 	size_t	len = stack->end - stack->head;
 	size_t	toff = stack->tail - stack->head;
@@ -58,7 +63,13 @@ stack_push(NStack stack, const char *name, VALUE val, Hint hint) {
 	stack->tail = stack->head + toff;
 	stack->end = stack->head + len + STACK_INC;
     }
-    stack->tail->name = name;
+    if (NV_BUF_MAX <= nlen) {
+        stack->tail->name = ox_strndup(name, nlen);
+    } else {
+        strncpy(stack->tail->name_buf, name, nlen);
+        stack->tail->name_buf[nlen] = '\0';
+        stack->tail->name = stack->tail->name_buf;
+    }
     stack->tail->val = val;
     stack->tail->hint = hint;
     stack->tail->childCnt = 0;
@@ -77,6 +88,9 @@ inline static Nv
 stack_pop(NStack stack) {
     if (stack->head < stack->tail) {
 	stack->tail--;
+        if (stack->tail->name != stack->tail->name_buf) {
+            xfree((char*)(stack->tail->name));
+        }
 	return stack->tail;
     }
     return 0;

data/lib/ox/version.rb

@@ -1,5 +1,5 @@
 
 module Ox
   # Current version of the module.
-  VERSION = '2.14.12'
+  VERSION = '2.14.14'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ox
 version: !ruby/object:Gem::Version
-  version: 2.14.12
+  version: 2.14.14
 platform: ruby
 authors:
 - Peter Ohler
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-12-27 00:00:00.000000000 Z
+date: 2023-01-27 00:00:00.000000000 Z
 dependencies: []
 description: "A fast XML parser and object serializer that uses only standard C lib.\n\nOptimized
   XML (Ox), as the name implies was written to provide speed optimized\nXML handling.