chromium/src
0
Fork 0
Code Activity

Fix: Remove multiline statement check until fuzzer is updated

Browse Source 
This change removes a check that restricted SQL queries to a
single statement. While the Chromium codebase has been updated
to comply with this restriction, the fuzzers currently generate
multiline statements. This check will be re-enabled once the
fuzzers are fixed.

Follow-up on this CL:
  https://chromium-review.googlesource.com/c/chromium/src/+/6196597/12/sql/database.cc

Bug: 394644269, 40779018, 394891084
Change-Id: I015f438d3bf56ae4d3413b145fa29742797d0cf8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6239953
Commit-Queue: Anthony Vallée-Dubois <anthonyvd@chromium.org>
Reviewed-by: Anthony Vallée-Dubois <anthonyvd@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1416959}
This commit is contained in:
Etienne Bergeron
2025-02-06 12:24:42 -08:00
committed by Chromium LUCI CQ
parent 89e0b6a070
commit 402a083742
1 changed files with 0 additions and 2 deletions

2
sql/database.cc

@@ -1515,8 +1515,6 @@ SqliteResultCode Database::ExecuteAndReturnResultCode(
while (base::IsAsciiWhitespace(*sql)) { while (base::IsAsciiWhitespace(*sql)) {
sql++; sql++;
} }
// Do not use multi-statement with Execute(...).
CHECK(!*sql);
} }
// Most calls to Execute() modify the database. The main exceptions would be // Most calls to Execute() modify the database. The main exceptions would be