[Webkit-unassigned] [Bug 62129] New: Assertion failure in SQLiteDatabase::turnOnIncrementalAutoVacuum()

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=62129

           Summary: Assertion failure in
                    SQLiteDatabase::turnOnIncrementalAutoVacuum()
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: All
        OS/Version: All
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: WebCore Misc.
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: chung at rim.com


SQLiteDatabase::turnOnIncrementalAutoVacuum() runs the VACUUM command on a database and this will result in an assertion failure in SQLite's btree.c (found while using SQLite version 3.7.2):

assert( !pBt->pPage1 && !pBt->pCursor ); in sqlite3BtreeSetPageSize(...)

This can be fixed by finalizing the statement that reads the auto_vacuum mode before running the VACUUM command as follows:

diff --git a/WebCore/platform/sql/SQLiteDatabase.cpp b/WebCore/platform/sql/SQLiteDatabase.cpp
index e7cef38..9d1bb34 100644
--- a/WebCore/platform/sql/SQLiteDatabase.cpp
+++ b/WebCore/platform/sql/SQLiteDatabase.cpp
@@ -444,6 +444,7 @@ bool SQLiteDatabase::turnOnIncrementalAutoVacuum()
     SQLiteStatement statement(*this, "PRAGMA auto_vacuum");
     int autoVacuumMode = statement.getColumnInt(0);
     int error = lastError();
+    statement.finalize();

     // Check if we got an error while trying to get the value of the auto_vacuum flag.
     // If we got a SQLITE_BUSY error, then there's probably another transaction in

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.