diagnostics_channel: Channel::binding_data_ dangles after environment cleanup, crashing node:sqlite at exit
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 48/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 活発
- 技術スタック
- javascript, node.js, sqlite
調査の方向性
まず、提供された node:diagnostics_channel と node:sqlite の再現を実行し、次に Environment::RunCleanup() と SQLite のプロファイルコールバックを通じて Channel::binding_data_ を追跡します。同じ終了シナリオで修正を確認します。これには、明示的な close() を行わずにステップの途中で残されたステートメントも含めます。また、プロセスが SIGSEGV なしで終了することを確認します。
索引モデルが issue の本文から書いたものです。
説明
Version
v27.0.0-pre (2dfdb6a4206)
Platform
Darwin 25.6.0 arm64
Subsystem
diagnostics_channel, sqlite
What steps will reproduce the bug?
Channel holds its BindingData as a raw pointer that is never cleared, so any native holder that outlives environment cleanup reads a destroyed object. Here's a replication case using node:sqlite:
const dc = require('node:diagnostics_channel');
const { DatabaseSync } = require('node:sqlite');
dc.subscribe('sqlite.db.query', () => {});
const db = new DatabaseSync(':memory:');
db.exec('CREATE TABLE t(x)');
const ins = db.prepare('INSERT INTO t VALUES (?)');
for (let i = 0; i < 200; i++) ins.run(i);
// Start iterating and abandon it: the statement stays mid-step, so the
// finalize that happens at teardown fires SQLite's profile callback.
const it = db.prepare('SELECT * FROM t').iterate();
it.next();
globalThis.keepAlive = { db, it };
console.log('reached end of script');
How often does it reproduce? Is there a required condition?
Three conditions are required (all included in the example above):
- A subscriber on
sqlite.db.query, so the SQLite profile hook is installed. - A statement left mid-step at exit, so that the finalize during teardown has a started statement to profile.
- No explicit
close(), so the statement is finalized by the destructor chain afterEnvironment::RunCleanup()rather than before it.
What is the expected behavior? Why is that the expected behavior?
Exiting a process that used node:sqlite with a sqlite.db.query subscriber
shouldn't crash.
What do you see instead?
SIGSEGV, exit 139, after the script has finished.
- 主要言語
- JavaScript
- スター
- 122k
- フォーク
- 37.4k
- 平均マージ
- 4日 4時間
- マージ済み PR(30日)
- 276
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
nodejs/node のほかの issue
-
doc
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
-
build
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
-
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
-
feature request
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
似ている issue
-
ai-observability bug team/ai-observability
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
AllTheMods/ATM-10-L#19 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
SignalK/n2k-signalk#345 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
-
難易度 1/5 1〜3時間 初心者へのやさしさ 88/100
JuliaPluto/PlutoPlotly.jl#72 ·