Skip to content

[v22.x] deps: update sqlite to 3.53.4 - #66246

Open
RafaelGSS wants to merge 6 commits into
nodejs:v22.xfrom
RafaelGSS:fix/v22-sqlite-3.53.4
Open

RafaelGSS wants to merge 6 commits into
nodejs:v22.xfrom
RafaelGSS:fix/v22-sqlite-3.53.4

Conversation

@RafaelGSS

Copy link
Copy Markdown
Member

Update the SQLite amalgamation from 3.51.3 to 3.53.4, matching the version already used on main. This includes the upstream FTS5 bounds checks for CVE-2026-11822 and CVE-2026-11824.

@RafaelGSS RafaelGSS added the v22.x Issues that can be reproduced on v22.x or PRs targeting the v22.x-staging branch. label Sep 23, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/security-wg
  • @nodejs/sqlite

@nodejs-github-bot nodejs-github-bot added dependencies PRs that add, update, or configure Node.js dependencies. needs-ci PRs that need a full CI run. sqlite Issues and PRs related to the SQLite subsystem. labels Sep 23, 2026
@richardlau

Copy link
Copy Markdown
Member

@RafaelGSS Were we not able to cherry-pick the individual update sqlite commits from main/later staging branches?

nodejs-github-bot and others added 6 commits September 23, 2026 14:15
PR-URL: nodejs#62699
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
(cherry picked from commit 17d03b5)
Signed-off-by: RafaelGSS <rafael.nunu@hotmail.com>
PR-URL: nodejs#63217
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Jithil P Ponnan <jithil@outlook.com>
(cherry picked from commit a91d859)
Signed-off-by: RafaelGSS <rafael.nunu@hotmail.com>
Backport the SQLite session extension fix for corrupt changesets that
omit old values for primary-key columns. This avoids passing NULL to
sessionBindValue() while applying UPDATE changesets.

Refs: https://sqlite.org/src/info/e807d4e3798efd53
Signed-off-by: junius-sec <sksch323@naver.com>
PR-URL: nodejs#63525
Refs: https://hackerone.com/reports/3736889
Refs: sqlite/sqlite@b869ed6
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
(cherry picked from commit d8ac301)
Signed-off-by: RafaelGSS <rafael.nunu@hotmail.com>
PR-URL: nodejs#63774
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
(cherry picked from commit 7239e15)
Signed-off-by: RafaelGSS <rafael.nunu@hotmail.com>
PR-URL: nodejs#64180
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Gürgün Dayıoğlu <hey@gurgun.day>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
(cherry picked from commit e09891c)
Signed-off-by: RafaelGSS <rafael.nunu@hotmail.com>
PR-URL: nodejs#64745
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Edy Silva <edigleyssonsilva@gmail.com>
(cherry picked from commit ede6d6a)
Signed-off-by: RafaelGSS <rafael.nunu@hotmail.com>
@RafaelGSS

RafaelGSS commented Sep 23, 2026

Copy link
Copy Markdown
Member Author

Yes, the six commits cherry-pick cleanly onto v22.x. I'll rework this PR to preserve the original commits.

@RafaelGSS
RafaelGSS force-pushed the fix/v22-sqlite-3.53.4 branch from 1797741 to 077d788 Compare September 23, 2026 17:16
@RafaelGSS

Copy link
Copy Markdown
Member Author

I'm just opening it manually instead of relying on branch-diff + cherry-pick so we can tag it on nodejs-dep-vuln-assessment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies PRs that add, update, or configure Node.js dependencies. needs-ci PRs that need a full CI run. sqlite Issues and PRs related to the SQLite subsystem. v22.x Issues that can be reproduced on v22.x or PRs targeting the v22.x-staging branch.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants