tiny_tds 3.4.0-aarch64-linux-gnu → 3.4.1-aarch64-linux-gnu

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 23aa8f70d2087d7477559a1e4b447fe573ffbafb8c6cdca1bc9cdcbb18d5b2ea
-  data.tar.gz: 3976e98e70f20ee6b6621458cf64e9643e1cfe3119aa2bd31119a1b7d34ba172
+  metadata.gz: f0b48605b6ffae2d66dfc95df9e4d893bbc73e903e11d0ccdc8b702c8c04aff0
+  data.tar.gz: a0d1edf3a9ec7980d99f3263570c743e4dcd343c55a5da83be7428b2e8ad908b
 SHA512:
-  metadata.gz: 847160581ef1e7b53381f0440ca3b38fbb525feda2b2b27a39fc664683c5db501ec70f021e28052cf1d04de05642a8cb48f854b9f6a8c948a10f0d1f810fa97d
-  data.tar.gz: 939c51dfe08bfbde2ba493ada5183deac5ab451c32b926ba14b97a04a5922d1f982b984ce07f42576b893f36fa5766364cca628309d7b3acbc8385894cd5c6dd
+  metadata.gz: 7b21bb3a07aa79c8d917776152b953b1806dc2fc3a22b89257ad768a40b77bef6e245abfa46f8d475a165d17d56436527256c74175289ce3454e54ae9d1596c9
+  data.tar.gz: 666f4916e9b6a98345c68a647f462d64310a1b5a540b17614f8a53cadb4207830ed66037bfbb7f052fbe434419488e9bafebce6fa6a60041f189f53be9a5ca2e

data/CHANGELOG.md

@@ -1,5 +1,10 @@
 ## (unreleased)
 
+## 3.4.1
+
+* Register the C globals that hold Ruby classes and modules with the GC, so raising `TinyTds::Error` after GC compaction no longer crashes with a segmentation fault. Fixes #608.
+* Use freetds v1.5.19 and OpenSSL v3.6.5 for Windows and Linux builds.
+
 ## 3.4.0
 
 * Add Ruby 4.0 to the cross compile list

data/VERSION

@@ -1 +1 @@
-3.4.0
+3.4.1

data/ext/tiny_tds/client.c

@@ -541,6 +541,7 @@ static VALUE rb_tinytds_connect(VALUE self, VALUE opts)
 void init_tinytds_client()
 {
   cTinyTdsClient = rb_define_class_under(mTinyTds, "Client", rb_cObject);
+  rb_global_variable(&cTinyTdsClient);
   rb_define_alloc_func(cTinyTdsClient, allocate);
   /* Define TinyTds::Client Public Methods */
   rb_define_method(cTinyTdsClient, "tds_version", rb_tinytds_tds_version, 0);

data/ext/tiny_tds/extconsts.rb

@@ -1,8 +1,8 @@
 ICONV_VERSION = ENV["TINYTDS_ICONV_VERSION"] || "1.18"
 ICONV_SOURCE_URI = "http://ftp.gnu.org/pub/gnu/libiconv/libiconv-#{ICONV_VERSION}.tar.gz"
 
-OPENSSL_VERSION = ENV["TINYTDS_OPENSSL_VERSION"] || "3.6.0"
+OPENSSL_VERSION = ENV["TINYTDS_OPENSSL_VERSION"] || "3.6.5"
 OPENSSL_SOURCE_URI = "https://www.openssl.org/source/openssl-#{OPENSSL_VERSION}.tar.gz"
 
-FREETDS_VERSION = ENV["TINYTDS_FREETDS_VERSION"] || "1.5.10"
+FREETDS_VERSION = ENV["TINYTDS_FREETDS_VERSION"] || "1.5.19"
 FREETDS_SOURCE_URI = "http://www.freetds.org/files/stable/freetds-#{FREETDS_VERSION}.tar.bz2"

data/ext/tiny_tds/result.c

@@ -702,9 +702,12 @@ void init_tinytds_result()
 {
   /* Data Classes */
   cKernel = rb_const_get(rb_cObject, rb_intern("Kernel"));
+  rb_global_variable(&cKernel);
   cDate = rb_const_get(rb_cObject, rb_intern("Date"));
+  rb_global_variable(&cDate);
   /* Define TinyTds::Result */
   cTinyTdsResult = rb_define_class_under(mTinyTds, "Result", rb_cObject);
+  rb_global_variable(&cTinyTdsResult);
   rb_undef_alloc_func(cTinyTdsResult);
   /* Define TinyTds::Result Public Methods */
   rb_define_method(cTinyTdsResult, "fields", rb_tinytds_result_fields, 0);

data/ext/tiny_tds/tiny_tds_ext.c

@@ -8,8 +8,10 @@ VALUE mTinyTds, cTinyTdsError;
 
 void Init_tiny_tds()
 {
-  mTinyTds      = rb_define_module("TinyTds");
+  mTinyTds = rb_define_module("TinyTds");
+  rb_global_variable(&mTinyTds);
   cTinyTdsError = rb_const_get(mTinyTds, rb_intern("Error"));
+  rb_global_variable(&cTinyTdsError);
   init_tinytds_client();
   init_tinytds_result();
 }

data/test/gc_compaction_test.rb

@@ -0,0 +1,37 @@
+require "test_helper"
+require "open3"
+require "rbconfig"
+
+class GcCompactionTest < Minitest::Spec
+  describe "after GC compaction" do
+    it "raises TinyTds::Error instead of crashing when a connection fails" do
+      skip "GC compaction is not supported on this platform" if !GC.respond_to?(:verify_compaction_references)
+
+      # Ruby 3.2 renamed the double_heap: keyword to expand_heap:
+      heap_keyword =
+        if Gem::Version.new(RUBY_VERSION) >= Gem::Version.new("3.2")
+          "expand_heap"
+        else
+          "double_heap"
+        end
+
+      script = <<~RUBY
+        require "tiny_tds"
+
+        GC.verify_compaction_references(#{heap_keyword}: true, toward: :empty)
+
+        begin
+          TinyTds::Client.new(host: "127.0.0.1", port: 1, username: "unused", password: "unused", login_timeout: 1)
+        rescue TinyTds::Error
+          exit 0
+        end
+
+        exit 1
+      RUBY
+
+      output, status = Open3.capture2e(RbConfig.ruby, "-I", File.expand_path("../lib", __dir__), "-e", script)
+
+      assert status.success?, "expected TinyTds::Error to be raised, got #{status.inspect}:\n#{output}"
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: tiny_tds
 version: !ruby/object:Gem::Version
-  version: 3.4.0
+  version: 3.4.1
 platform: aarch64-linux-gnu
 authors:
 - Ken Collins
@@ -245,6 +245,7 @@ files:
 - test/bin/install-freetds.sh
 - test/bin/restore-from-native-gem.ps1
 - test/client_test.rb
+- test/gc_compaction_test.rb
 - test/gem_test.rb
 - test/result_test.rb
 - test/schema/1px.gif