Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(146)

Unified Diff: third_party/sqlite/fts3_85522.patch

Issue 885473002: [sql] Rewrite sqlite patching "system". (Closed) Base URL: http://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/sqlite/fts3_85522.patch
diff --git a/third_party/sqlite/fts3_85522.patch b/third_party/sqlite/fts3_85522.patch
deleted file mode 100644
index 8d7d1813795a075c4b5ee658a76e1d9172c41613..0000000000000000000000000000000000000000
--- a/third_party/sqlite/fts3_85522.patch
+++ /dev/null
@@ -1,21 +0,0 @@
-Fix http://crbug.com/85522
-
-WebDatabase uses an authorizer to prevent inappropriate access.
-fts3.c uses 'PRAGMA page_size' to tune the query optimizer, but PRAGMA
-is on the disallowed list. This patch adds a default return value for
-SQLITE_AUTH failures.
-
-diff --git src/ext/fts3/fts3.c src/ext/fts3/fts3.c
-index 8498cfa..dfa3891 100644
---- src/ext/fts3/fts3.c
-+++ src/ext/fts3/fts3.c
-@@ -630,6 +630,9 @@ static void fts3DatabasePageSize(int *pRc, Fts3Table *p){
- sqlite3_step(pStmt);
- p->nPgsz = sqlite3_column_int(pStmt, 0);
- rc = sqlite3_finalize(pStmt);
-+ }else if( rc==SQLITE_AUTH ){
-+ p->nPgsz = 1024;
-+ rc = SQLITE_OK;
- }
- }
- assert( p->nPgsz>0 || rc!=SQLITE_OK );

Powered by Google App Engine
This is Rietveld 408576698