/ Check-in [b7905b8c]
Login

Many hyperlinks are disabled.
Use anonymous login to enable hyperlinks.

Overview
Comment:When closing a (shared-cache) database connection, be sure to clear out all KeyInfo objects cached on Index objects. Fix for ticket [e4a18565a36884b00edf].
Downloads: Tarball | ZIP archive | SQL archive
Timelines: family | ancestors | descendants | both | branch-3.8.7
Files: files | file ages | folders
SHA1: b7905b8c5fe266f43c99a44c6b9aa4b56b2ba4d1
User & Date: drh 2014-12-05 14:51:22
Context
2014-12-05
14:54
Increase the version number to 3.8.7.3. check-in: 9f7560bc user: drh tags: branch-3.8.7
14:51
When closing a (shared-cache) database connection, be sure to clear out all KeyInfo objects cached on Index objects. Fix for ticket [e4a18565a36884b00edf]. check-in: b7905b8c user: drh tags: branch-3.8.7
14:44
When closing a (shared-cache) database connection, be sure to clear out all KeyInfo objects cached on Index objects. Fix for ticket [e4a18565a36884b00edf]. check-in: adca7688 user: drh tags: trunk
2014-12-04
16:29
If a table is the right operand of a LEFT JOIN, then any column of that table can be NULL even if that column has a NOT NULL constraint. Fix for ticket [6f2222d550f5b0ee7ed]. check-in: 5a80957b user: drh tags: branch-3.8.7
Changes
Hide Diffs Side-by-Side Diffs Ignore Whitespace Patch

Changes to src/main.c.

   928    928   
   929    929     /* Close all database connections */
   930    930     for(j=0; j<db->nDb; j++){
   931    931       struct Db *pDb = &db->aDb[j];
   932    932       if( pDb->pBt ){
   933    933         sqlite3BtreeClose(pDb->pBt);
   934    934         pDb->pBt = 0;
   935         -      if( j!=1 ){
          935  +      if( j!=1 && pDb->pSchema ){
          936  +        /* Must clear the KeyInfo cache.  See ticket [e4a18565a36884b00edf] */
          937  +        for(i=sqliteHashFirst(&pDb->pSchema->idxHash); i; i=sqliteHashNext(i)){
          938  +          Index *pIdx = sqliteHashData(i);
          939  +          sqlite3KeyInfoUnref(pIdx->pKeyInfo);
          940  +          pIdx->pKeyInfo = 0;
          941  +        }
   936    942           pDb->pSchema = 0;
   937    943         }
   938    944       }
   939    945     }
   940    946     /* Clear the TEMP schema separately and last */
   941    947     if( db->aDb[1].pSchema ){
   942    948       sqlite3SchemaClear(db->aDb[1].pSchema);

Added test/sharedB.test.

            1  +# 2014-12-05
            2  +#
            3  +# The author disclaims copyright to this source code.  In place of
            4  +# a legal notice, here is a blessing:
            5  +#
            6  +#    May you do good and not evil.
            7  +#    May you find forgiveness for yourself and forgive others.
            8  +#    May you share freely, never taking more than you give.
            9  +#
           10  +#***********************************************************************
           11  +#
           12  +# Open two database connections on the same database in shared cache
           13  +# mode.  Hold one open while repeatedly closing, reopening, and using
           14  +# the second.
           15  +#
           16  +# This test is designed to demostrate that the fix for ticket
           17  +# [e4a18565a36884b00edf66541f38c693827968ab] works.  
           18  +#
           19  +
           20  +
           21  +set testdir [file dirname $argv0]
           22  +source $testdir/tester.tcl
           23  +if {[run_thread_tests]==0} { finish_test ; return }
           24  +db close
           25  +set ::testprefix sharedB
           26  +
           27  +set ::enable_shared_cache [sqlite3_enable_shared_cache 1]
           28  +
           29  +#-------------------------------------------------------------------------
           30  +#
           31  +do_test 1.1 {
           32  +  sqlite3 db1 test.db
           33  +  sqlite3 db2 test.db
           34  +
           35  +  db1 eval {
           36  +    CREATE TABLE t1(x,y TEXT COLLATE nocase);
           37  +    WITH RECURSIVE
           38  +      c(i) AS (VALUES(1) UNION ALL SELECT i+1 FROM c WHERE i<100)
           39  +    INSERT INTO t1(x,y) SELECT i, printf('x%03dy',i) FROM c;
           40  +    CREATE INDEX t1yx ON t1(y,x);
           41  +  }
           42  +  db2 eval {
           43  +    SELECT x FROM t1 WHERE y='X014Y';
           44  +  }
           45  +} {14}
           46  +
           47  +for {set j 1} {$j<=100} {incr j} {
           48  +  do_test 1.2.$j {
           49  +    db2 close
           50  +    sqlite3 db2 test.db
           51  +    db2 eval {
           52  +      SELECT x FROM t1 WHERE y='X014Y';
           53  +    }
           54  +  } {14}
           55  +}
           56  +
           57  +db1 close
           58  +db2 close
           59  +sqlite3_enable_shared_cache $::enable_shared_cache
           60  +finish_test