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

Unified Diff: third_party/sqlite/sqlite-poison.patch

Issue 6823057: Cleanup SQLite 3.6.18 import. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years, 8 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
« no previous file with comments | « third_party/sqlite/safe-tolower.patch ('k') | third_party/sqlite/src/LICENSE » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/sqlite/sqlite-poison.patch
diff --git a/third_party/sqlite/sqlite-poison.patch b/third_party/sqlite/sqlite-poison.patch
deleted file mode 100644
index 38f81a0c52fa2ff59b0e536f2418b4e003f97644..0000000000000000000000000000000000000000
--- a/third_party/sqlite/sqlite-poison.patch
+++ /dev/null
@@ -1,98 +0,0 @@
-Index: src/btree.c
-===================================================================
---- src/btree.c 2009-09-09 06:45:19.000000000 -0700
-+++ src/btree.c 2009-09-14 18:17:53.000000000 -0700
-@@ -24,6 +24,12 @@
- static const char zMagicHeader[] = SQLITE_FILE_HEADER;
-
- /*
-+** The header string that appears at the beginning of a SQLite
-+** database which has been poisoned.
-+*/
-+static const char zPoisonHeader[] = "SQLite poison 3";
-+
-+/*
- ** Set this global variable to 1 to enable tracing using the TRACE
- ** macro.
- */
-@@ -2337,6 +2343,7 @@
- if( rc ) return rc;
- memcpy(data, zMagicHeader, sizeof(zMagicHeader));
- assert( sizeof(zMagicHeader)==16 );
-+ assert( sizeof(zMagicHeader)==sizeof(zPoisonHeader) );
- put2byte(&data[16], pBt->pageSize);
- data[18] = 1;
- data[19] = 1;
-@@ -7804,4 +7811,72 @@
- assert(!pCur->aOverflow);
- pCur->isIncrblobHandle = 1;
- }
-+
-+/* Poison the db so that other clients error out as quickly as
-+** possible.
-+*/
-+int sqlite3Poison(sqlite3 *db){
-+ int rc;
-+ Btree *p;
-+ BtShared *pBt;
-+ unsigned char *pP1;
-+
-+ if( db == NULL) return SQLITE_OK;
-+
-+ /* Database 0 corrosponds to the main database. */
-+ if( db->nDb<1 ) return SQLITE_OK;
-+ p = db->aDb[0].pBt;
-+ pBt = p->pBt;
-+
-+ /* If in a transaction, roll it back. Committing any changes to a
-+ ** corrupt database may mess up evidence, we definitely don't want
-+ ** to allow poisoning to be rolled back, and the database is anyhow
-+ ** going bye-bye RSN.
-+ */
-+ /* TODO(shess): Figure out if this might release the lock and let
-+ ** someone else get in there, which might deny us the lock a couple
-+ ** lines down.
-+ */
-+ if( sqlite3BtreeIsInTrans(p) ) sqlite3BtreeRollback(p);
-+
-+ /* Start an exclusive transaction. This will check the headers, so
-+ ** if someone else poisoned the database we should get an error.
-+ */
-+ rc = sqlite3BtreeBeginTrans(p, 2);
-+ /* TODO(shess): Handle SQLITE_BUSY? */
-+ if( rc!=SQLITE_OK ) return rc;
-+
-+ /* Copied from sqlite3BtreeUpdateMeta(). Writing the old version of
-+ ** the page to the journal may be overkill, but it probably won't
-+ ** hurt.
-+ */
-+ assert( pBt->inTrans==TRANS_WRITE );
-+ assert( pBt->pPage1!=0 );
-+ rc = sqlite3PagerWrite(pBt->pPage1->pDbPage);
-+ if( rc ) goto err;
-+
-+ /* "SQLite format 3" changes to
-+ ** "SQLite poison 3". Be extra paranoid about making this change.
-+ */
-+ if( sizeof(zMagicHeader)!=16 ||
-+ sizeof(zPoisonHeader)!=sizeof(zMagicHeader) ){
-+ rc = SQLITE_ERROR;
-+ goto err;
-+ }
-+ pP1 = pBt->pPage1->aData;
-+ if( memcmp(pP1, zMagicHeader, 16)!=0 ){
-+ rc = SQLITE_CORRUPT;
-+ goto err;
-+ }
-+ memcpy(pP1, zPoisonHeader, 16);
-+
-+ /* Push it to the database file. */
-+ return sqlite3BtreeCommit(p);
-+
-+ err:
-+ /* TODO(shess): What about errors, here? */
-+ sqlite3BtreeRollback(p);
-+ return rc;
-+}
-+
- #endif
« no previous file with comments | « third_party/sqlite/safe-tolower.patch ('k') | third_party/sqlite/src/LICENSE » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698