GitHub Advisoryadvisory · crates.io

GHSA-q89g-4vhh-mvvm Incorrect Lifetime Bounds on Closures in `rusqlite`

HIGH Published 2022-06-17 Modified 2022-06-17 Official page ↗
Ecosystem
crates.io
Packages
rusqlite
Severity
High
Reviewed
2022-06-17

Affected packages

• crates.io rusqlite: >= 0.26.0, < 0.26.2 → fixed in 0.26.2 • crates.io rusqlite: >= 0.25.0, < 0.25.4 → fixed in 0.25.4

Description

The lifetime bound on several closure-accepting `rusqlite` functions (specifically, functions which register a callback to be later invoked by SQLite) was too relaxed. If a closure referencing borrowed values on the stack is was passed to one of these functions, it could allow Rust code to access objects on the stack after they have been dropped. The impacted functions are: • Under `cfg(feature = "functions")`: `Connection::create_scalar_function`, `Connection::create_aggregate_function` and `Connection::create_window_function`. • Under `cfg(feature = "hooks")`: `Connection::commit_hook`, `Connection::rollback_hook` and `Connection::update_hook`. • Under `cfg(feature = "collation")`: `Connection::create_collation`. The issue exists in all `0.25.*` versions prior to `0.25.4`, and all `0.26.*` versions prior to 0.26.2 (specifically: `0.25.0`, `0.25.1`, `0.25.2`, `0.25.3`, `0.26.0`, and `0.26.1`). The fix is available in versions `0.26.2` and newer, and also has been back-ported to `0.25.4`. As it does not exist in `0.24.*`, all affected versions should have an upgrade path to a semver-compatible release.

References