Merge pull request #556 from jhawthorn/fix_write_barrier

Use write barrier when setting busy_handler
sparklemotion · Sep 12, 2024 · 480f3e7 · 480f3e7
2 parents 164485a + f759e82
commit 480f3e7
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/ext/sqlite3/database.c b/ext/sqlite3/database.c
@@ -246,7 +246,7 @@ busy_handler(int argc, VALUE *argv, VALUE self)
     rb_scan_args(argc, argv, "01", &block);
 
     if (NIL_P(block) && rb_block_given_p()) { block = rb_block_proc(); }
-    ctx->busy_handler = block;
+    RB_OBJ_WRITE(self, &ctx->busy_handler, block);
 
     status = sqlite3_busy_handler(
                  ctx->db,

diff --git a/lib/sqlite3/database.rb b/lib/sqlite3/database.rb
@@ -127,7 +127,6 @@ def initialize file, options = {}, zvfs = nil
 
       @tracefunc = nil
       @authorizer = nil
-      @busy_handler = nil
       @progress_handler = nil
       @collations = {}
       @functions = {}