GHSA-8cpq-38p9-67gx: Kysely has a MySQL SQL Injection via Insufficient Backslash Escaping in `sql.lit(string)` usage or similar methods that append string literal values into the compiled SQL strings
SummaryKysely's `DefaultQueryCompiler.sanitizeStringLiteral()` only escapes single quotes by doubling them (`'` → `''`) but does not escape backslashes. When used with the MySQL dialect (where `NO_BACKSLASH_ESCAPES` is OFF by default), an attacker can use a backslash to escape the trailing quote of a string literal, breaking out of the string context and injecting arbitrary SQL. This affects any code path that uses `ImmediateValueTransformer` to inline values — specifically `CreateIndexBuilder.where()` and `CreateViewBuilder.as()`.DetailsThe root cause is in `DefaultQueryCompiler.sanitizeStringLiteral()`:**`src/query-compiler/default-query-compiler.ts:1819-1821`**Where `LIT_WRAP_REGEX` is defined as `/'/g` (line 121). This only doubles single quotes — it does not escape backslash characters.The function is called from `appendStringLiteral()` which wraps the sanitized value in single quotes:**`src/query-compiler/default-query-compiler.ts:1841-1845`**This is reached when `visitValue()` encounters an immediate value node (line 525-527), which is created by `ImmediateValueTransformer` used in `CreateIndexBuilder.where()`:**`src/schema/create-index-builder.ts:266-278`**The `MysqlQueryCompiler` (at `src/dialect/mysql/mysql-query-compiler.ts:6-75`) extends `DefaultQueryCompiler` but does **not** override `sanitizeStringLiteral`, inheriting the backslash-unaware implementation.**Exploitation mechanism:**In MySQL with the default `NO_BACKSLASH_ESCAPES=OFF` setting, the backslash character (`\`) acts as an escape character inside string literals. Given input `\' OR 1=1 --`:`sanitizeStringLiteral` doubles the quote: `\'' OR 1=1 --``appendStringLiteral` wraps: `'\'' OR 1=1 --'`MySQL interprets `\'` as an escaped (literal) single quote, so the string content is `'` and the second `'` closes the string` OR 1=1 --` is parsed as SQLPoCTo verify against a live MySQL instance:Impact**SQL Injection:** An attacker who controls values passed to `CreateIndexBuilder.where()` or `CreateViewBuilder.as()` can inject arbitrary SQL statements when the application uses the MySQL dialect.**Data Exfiltration:** Injected SQL can read arbitrary data from the database using UNION-based or subquery-based techniques.**Data Modification/Destruction:** Stacked queries or subqueries can modify or delete data.**Authentication Bypass:** If index creation or view definitions are influenced by user input in application logic, the injection can alter query semantics to bypass access controls.The attack complexity is rated High (AC:H) because exploitation requires an application to pass untrusted user input into DDL schema builder methods, which is an atypical but not impossible usage pattern. The `CreateIndexBuilder.where()` docstring (line 247) notes "Parameters are always sent as literals due to database restrictions" without warning about the security implications.Recommended Fix`MysqlQueryCompiler` should override `sanitizeStringLiteral` to escape backslashes before doubling quotes:**`src/dialect/mysql/mysql-query-compiler.ts`**Alternatively, the library could use parameterized queries for these DDL builders where the database supports it, avoiding string literal interpolation entirely. For databases that don't support parameters in DDL statements, the dialect-specific compiler must escape all characters that have special meaning in that dialect's string literal syntax.
CVSS v3.1: 8.1