rage-iodine 6.1.0 → 6.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6096c74f8f76040521a3d8d977be278e9da518a65e3a5dedd7f715ebcf8c8ae7
-  data.tar.gz: 365e0c9732780d5672f3ca2e9b4ca23cfe1a954cb24a2ec58d013e2a59081e1f
+  metadata.gz: 5bf4dfb83b39ffd7e2c88b535342d69b9ba6492dd80381e747c7f8909025b5c1
+  data.tar.gz: 7d2e6bdafc532d94b296fb0743cfe832942421ecfa8ddab78a21f7f3e5c50f1e
 SHA512:
-  metadata.gz: 8a8fc73f923a749da72f025797613c4036f79e1405e4f8791b8276a8629cdb511d8f9ec40e1b56087f5bc049058351fd7d126372a179a228de4de77aa3a295b9
-  data.tar.gz: 19526c5ba722bebb05e18cb4125f7cf46acd8c129f5766709888718fe45f6876065fb9fd34f5c98ec8ccfa86a962418b607e63fbf4c249bbc9419ce6cb1ef053
+  metadata.gz: 9989075b63db053de7dd6678cb317aef338c9150acfb88da70a8cf0d77f13acd1b6c5aff12cadf212a85534f84832fb9b2af71580f88eb5fcc1de065f2d3b015
+  data.tar.gz: bd74de0e0b28d2b035cc3df029ea7930ef57221ab4a443d88df873421ce4634567cb7e2fb06cfaec9a3e878feacb0d70541c6710191855c73b6d55c6173daf10

data/.github/workflows/release.yml

@@ -4,6 +4,7 @@ on:
   push:
     branches:
       - master
+      - release/*
     paths:
       - lib/iodine/version.rb
 

data/CHANGELOG.md

@@ -6,10 +6,26 @@ Please notice that this change log contains changes for upcoming releases as wel
 
 ## Changes:
 
+#### Change log v.6.3.0 (2026-10-23)
+
+**Fix**: Fix `rack.input` buffer coderange
+
+#### Change log v.5.9.0/v.6.2.0 (2026-10-15)
+
+**Fix**: Prevent process hang on exit caused by a use-after-free in pub/sub teardown (`fio_cluster_at_exit` read a channel freed by `fio_unsubscribe` of its last subscription; it now performs a bounded best-effort cleanup instead). See #21.
+
+#### Change log v.5.8.0 (2026-10-09)
+
+**Update**: Encode parameters as UTF-8
+
 #### Change log v.6.1.0 (2026-10-04)
 
 **Update**: Refactor parameters parsing
 
+#### Change log v.5.7.0 (2026-10-04)
+
+**Update**: Refactor parameters parsing
+
 #### Change log v.6.0.0 (2026-09-06)
 
 **Update**: Update scheduler `read`/`write` hooks

data/ext/iodine/fio.c

@@ -7658,6 +7658,25 @@ static inline void fio_cluster_inform_root_about_channel(channel_s *ch,
 /* *****************************************************************************
  * Initialization
  **************************************************************************** */
+/* best-effort teardown without fio_unsubscribe: it may free `ch` (and remove
+ * it from the set) while this loop still uses it (#21) */
+FIO_FUNC void fio_pubsub_at_exit_clear(fio_ch_set_s *channels) {
+  while (fio_ch_set_count(channels)) {
+    channel_s *ch = fio_ch_set_last(channels);
+    while (fio_ls_embd_any(&ch->subscriptions)) {
+      subscription_s *sub =
+          FIO_LS_EMBD_OBJ(subscription_s, node, ch->subscriptions.next);
+      fio_ls_embd_remove(&sub->node);
+      sub->unsubscribed = 1;
+      sub->on_message = NULL;
+      sub->on_unsubscribe = NULL;
+      fio_subscription_free(sub);
+    }
+    fio_ch_set_pop(channels);
+  }
+  fio_ch_set_free(channels);
+}
+
 #ifndef __MINGW32__
 static void fio_accept_after_fork(void *ignore) {
   /* prevent `accept` backlog in parent */
@@ -7671,38 +7690,9 @@ static void fio_cluster_at_exit(void *ignore) {
   /* unlock all */
   fio_pubsub_on_fork();
   /* clear subscriptions of all types */
-  while (fio_ch_set_count(&fio_postoffice.patterns.channels)) {
-    channel_s *ch = fio_ch_set_last(&fio_postoffice.patterns.channels);
-    while (fio_ls_embd_any(&ch->subscriptions)) {
-      subscription_s *sub =
-          FIO_LS_EMBD_OBJ(subscription_s, node, ch->subscriptions.next);
-      fio_unsubscribe(sub);
-    }
-    fio_ch_set_pop(&fio_postoffice.patterns.channels);
-  }
-
-  while (fio_ch_set_count(&fio_postoffice.pubsub.channels)) {
-    channel_s *ch = fio_ch_set_last(&fio_postoffice.pubsub.channels);
-    while (fio_ls_embd_any(&ch->subscriptions)) {
-      subscription_s *sub =
-          FIO_LS_EMBD_OBJ(subscription_s, node, ch->subscriptions.next);
-      fio_unsubscribe(sub);
-    }
-    fio_ch_set_pop(&fio_postoffice.pubsub.channels);
-  }
-
-  while (fio_ch_set_count(&fio_postoffice.filters.channels)) {
-    channel_s *ch = fio_ch_set_last(&fio_postoffice.filters.channels);
-    while (fio_ls_embd_any(&ch->subscriptions)) {
-      subscription_s *sub =
-          FIO_LS_EMBD_OBJ(subscription_s, node, ch->subscriptions.next);
-      fio_unsubscribe(sub);
-    }
-    fio_ch_set_pop(&fio_postoffice.filters.channels);
-  }
-  fio_ch_set_free(&fio_postoffice.filters.channels);
-  fio_ch_set_free(&fio_postoffice.patterns.channels);
-  fio_ch_set_free(&fio_postoffice.pubsub.channels);
+  fio_pubsub_at_exit_clear(&fio_postoffice.patterns.channels);
+  fio_pubsub_at_exit_clear(&fio_postoffice.pubsub.channels);
+  fio_pubsub_at_exit_clear(&fio_postoffice.filters.channels);
 
   /* clear engines */
   FIO_PUBSUB_DEFAULT = FIO_PUBSUB_CLUSTER;
@@ -12106,6 +12096,23 @@ FIO_FUNC void fio_pubsub_test(void) {
   ++expect;
   fio_defer_perform();
   FIO_ASSERT(counter == expect, "unsubscribe wasn't called for named channel!");
+
+  /* Regression #21: at-exit cleanup must terminate and empty the set, even
+   * with several subscriptions per channel or an already-marked one. */
+  fio_subscribe(.channel = {0, 9, "hang-test"}, .udata1 = &counter,
+                .on_message = fio_pubsub_test_on_message,
+                .on_unsubscribe = fio_pubsub_test_on_unsubscribe);
+  s = fio_subscribe(.channel = {0, 9, "hang-test"}, .udata1 = &counter,
+                    .on_message = fio_pubsub_test_on_message,
+                    .on_unsubscribe = fio_pubsub_test_on_unsubscribe);
+  FIO_ASSERT(s, "fio_subscribe FAILED for hang-test channel.");
+  fio_trylock(&s->unsubscribed);
+  fio_pubsub_at_exit_clear(&fio_postoffice.pubsub.channels);
+  FIO_ASSERT(!fio_ch_set_count(&fio_postoffice.pubsub.channels),
+             "at-exit cleanup left channels behind (#21)");
+  fio_defer_perform();
+  FIO_ASSERT(counter == expect, "at-exit cleanup shouldn't call callbacks!");
+
   fio_data->is_worker = 0;
   fio_data->active = 0;
   fio_data->workers = 0;

data/ext/iodine/http.c

@@ -68,6 +68,7 @@ static VALUE cUploadedFile;
 static ID create_id;
 static ID path_id;
 static rb_encoding *IodineBinaryEncoding;
+static rb_encoding *IodineUTF8Encoding;
 
 /* *****************************************************************************
 Small Helpers
@@ -2010,7 +2011,7 @@ static void http_mime_parser_on_data(http_mime_parser_s *parser, void *name,
                                      size_t value_len) {
   // for regular values - just add them to params
   if (!filename_len) {
-    VALUE r_value = rb_enc_str_new(value, value_len, IodineBinaryEncoding);
+    VALUE r_value = rb_enc_str_new(value, value_len, IodineUTF8Encoding);
     http_add_to_params(http_mime_parser2fio(parser)->params, name, name_len, r_value);
     return;
   }
@@ -2085,7 +2086,7 @@ static void http_mime_parser_on_partial_end(http_mime_parser_s *parser) {
 
   FIOBJ slice = fiobj_data_slice(fio_parser->h->body, fio_parser->partial_offset, fio_parser->partial_length);
   fio_str_info_s s_slice = fiobj_obj2cstr(slice);
-  VALUE r_value = rb_enc_str_new(s_slice.data, s_slice.len, IodineBinaryEncoding);
+  VALUE r_value = rb_enc_str_new(s_slice.data, s_slice.len, IodineUTF8Encoding);
   http_add_to_params(fio_parser->params, RSTRING_PTR(fio_parser->partial_name), RSTRING_LEN(fio_parser->partial_name), r_value);
 
   fio_parser->partial_name = Qnil;
@@ -2701,6 +2702,7 @@ void http_init(void) {
   path_id = rb_intern("path");
 
   IodineBinaryEncoding = rb_enc_find("binary");
+  IodineUTF8Encoding = rb_enc_find("UTF-8");
 }
 
 /* *****************************************************************************

data/ext/iodine/iodine_helpers.c

@@ -242,7 +242,7 @@ Convert query string into a Ruby object without recursion.
 */
 static VALUE build_query_component(char *str, size_t len,
                                     uint8_t should_decode) {
-  VALUE result = rb_str_new(str, len);
+  VALUE result = rb_enc_str_new(str, len, IodineUTF8Encoding);
   if (should_decode) {
     url_decode_inplace(Qnil, result);
   }
@@ -290,7 +290,7 @@ static VALUE parse_nested_query_internal(char *str, size_t len) {
                                      pair_end - separator - 1,
                                      value_should_decode);
     } else if (key_len > 1 && memchr(key_data + 1, '[', key_len - 1)) {
-      value = rb_str_new(NULL, 0);
+      value = rb_enc_str_new(NULL, 0, IodineUTF8Encoding);
     } else {
       value = Qnil;
     }

data/ext/iodine/iodine_rack_io.c

@@ -146,6 +146,7 @@ static VALUE rio_read(int argc, VALUE *argv, VALUE self) {
     } else {
       // make sure the buffer is binary encoded.
       rb_enc_associate(buffer, IodineBinaryEncoding);
+      rb_str_modify(buffer);
       if (rb_str_capacity(buffer) < (size_t)buf.len)
         rb_str_resize(buffer, buf.len);
       memcpy(RSTRING_PTR(buffer), buf.data, buf.len);

data/lib/iodine/version.rb

@@ -1,3 +1,3 @@
 module Iodine
-  VERSION = '6.1.0'.freeze
+  VERSION = '6.3.0'.freeze
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rage-iodine
 version: !ruby/object:Gem::Version
-  version: 6.1.0
+  version: 6.3.0
 platform: ruby
 authors:
 - Boaz Segev