Simplifications and tweaks to vdbeaux.c resulting from structural testing. (CVS 6891)

FossilOrigin-Name: fa49666fb913f0d82e84bdfa2af3a294be04e47c

Make sure the IN operator works with zeroblobs. Ticket #3965.
Other simplifications associated with structural testing. (CVS 6890)

FossilOrigin-Name: 25dd342283046aaf66a679348ef1c7364c616402

Remove the SQLITE_CommitBusy flag.  This was an attempt to block recursion
on the sqlite3_commit_hook() interface.  But such recursion is explicitly
disallowed, so the flag is pointless. (CVS 6889)

FossilOrigin-Name: 1c2bfc43a4fd5b779a3b5b5b8ca5b41cb7250b5a

Change an unreachable condition in btree.c to a NEVER(). (CVS 6888)

FossilOrigin-Name: 9f800e11391a108d7aa57f5a96d9a58711129132

In sqlite3PagerWrite(), do not set the PGHDR_NEED_SYNC flag on a page if an IO error occured while attempting to journal it. (CVS 6887)

FossilOrigin-Name: b9be365d85fddedbfa93eebf3ee62a140cbaa426

Remove an assert ("assert( subpage>0 )") from btree.c that may not be true for a corrupt database. Also add comments and other assert() statements to btree.c function moveToRoot(). (CVS 6886)

FossilOrigin-Name: 3151dab9c78106217ec80ebadc666dfd11b42029

Remove a case from BtreeMovetoUnpacked() that is unreachable as of (6881). (CVS 6885)

FossilOrigin-Name: 39ce2097da03176e256a2ff35bb857e578f3ca2d

Simplifications to btree.c to help facilitate coverage testing. (CVS 6884)

FossilOrigin-Name: dfb146ef003bae8ff216baf68cc43d7eb7855c68

When an OOM fault occurs while moving a page for autovacuum in an in-memory
database, be sure to undo the page move prior to returning the error, to
avoid corrupting the in-memory database file. (CVS 6883)

FossilOrigin-Name: d0964b93669123e228e7ce1890167447c56753b5

Remove an unused parameter from the accessPayload() function in btree.c. (CVS 6882)

FossilOrigin-Name: 7deb6568d89335926b77336756837c6dc3985529

Fix a case where deleting a row from a corrupt database could cause an assert to fail. (CVS 6881)

FossilOrigin-Name: 6994b41a94a60f6460cf9814767db321ab3851f7

Bug fix in the computation of the number of pages to autovacuum when
nReserve is greater than zero. (CVS 6880)

FossilOrigin-Name: 618a83d65f973183d21245721dc656a35ff594a4

Unwind some complex conditions in sqlite3BtreeDelete() into
separate "if" statements. (CVS 6879)

FossilOrigin-Name: d99bde9ca61eeccfe6363ff0882fd4bcdb9a34dc

Remove another unreachable branch from btree.c. (CVS 6878)

FossilOrigin-Name: b0853100a9f8e185e8d027502822337a79a2ba0c

Modify the integrity-check code so that each b-tree page inspected is reinitialized while doing so, even if an initialized copy exists in the cache. This prevents an assert from failing when running integrity_check on a corrupt database. (CVS 6877)

FossilOrigin-Name: 709576c670f802bf4b6e5c0e8db2bbde2cc16a90

Remove unreachable condition from btree.c. (CVS 6876)

FossilOrigin-Name: 47b40fefa67f7c563ce2004509aaf8e203038be4

Fix a pager reference count leak in btree that occurs on an OOM following
a database page size change. (CVS 6875)

FossilOrigin-Name: c6dfc8bd3911b4c93969bfc13d9931965feed674

Fix a double-free that can occur when using the fts3 legacy syntax '-' operator. Add tests for the same operator. Ticket #3960. (CVS 6874)

FossilOrigin-Name: c19d419e8cf94a26d9bb6ad478e84841168a882e

Simplifications to the PRAGMA integrity_check logic in btree.c. (CVS 6873)

FossilOrigin-Name: 440c573c7e2a22f9a67a9571883e205fbadb7c11

Fix two faulty assert() statements btree.  (It was possible for the assertions
to be false given a corrupt database file as input.) (CVS 6872)

FossilOrigin-Name: 9403f04dfdabdd2ba9adb1a91ca9b400cf4ca6b3

Remove an unused branch from btree.c. (CVS 6871)

FossilOrigin-Name: 45efb78a8a1e01a0154153b10469bb34d6d129eb

In btree.c, have setSharedCacheLock() assume that the b-tree is sharable instead of checking to see if it is. The caller has already performed this check. (CVS 6870)

FossilOrigin-Name: 3190eefd5f5285a5250d6f4729f109852fdb02ab

Restore (btree_set_cache_size) to test3.c. This is used on unix for the tcl "crash tests". Make some functions no longer used outside of btree.c static. (CVS 6869)

FossilOrigin-Name: 1d96ce8c76e007ca29e7971a6c72b6387fe0fb93

removed the test function sqlite3BtreeFlags() (test code only); (CVS 6868)

FossilOrigin-Name: 579ba6c83f52d26860e5152f06355aa4f49e568d

removed unused functions from test3.c (test code only); (CVS 6867)

FossilOrigin-Name: 0eb69e8dda13e2d52a06f66a71480f59fd766271

Simplifications and additional testcase() macros for btree.c. (CVS 6866)

FossilOrigin-Name: 1b8c3a8246a5d8b8f5607014f68483735a403249

Fix a typo in (6861): "ifndef SQLITE_TEST" -> "ifdef SQLITE_TEST". (CVS 6865)

FossilOrigin-Name: 890dbab28c3a1af4a1e01221318c9200574139e7

Revert part of (6860) that was accidentally checked in. (CVS 6864)

FossilOrigin-Name: 11e295ccd28fe669ee74c846db507040895a5e5b

Add another boundary-condition testcase() macro to btree.c.  Correct the
comment on modifyPagePointer(). (CVS 6863)

FossilOrigin-Name: fed66e8695ee552d5ca05ac91485cfc5940e7496

Improvements to comments in btree.c:freeSpace().  Added a testcase() macro
to verify a boundary case is tested in sqlite3BtreeInitPage(). (CVS 6862)

FossilOrigin-Name: 5ac991b3cea214095b8369e85f3e0ad8e5059ad6

Comment out sqlite3BtreeFlags() when not in debug mode.  Tweaks to
database corruption detection logic in freeSpace(). (CVS 6861)

FossilOrigin-Name: 196cbf82290a308e78b13044d0fae993f2d5cb90

Fix error handling in sqlite3BtreePutData(). (CVS 6860)

FossilOrigin-Name: 86eba4f16fd9e97344ab6dfb9cb9a85cf14751dd

Remove an ALWAYS clause that could be false if an IO error within a statement executed by a virtual table caused an emergency rollback. (CVS 6859)

FossilOrigin-Name: 0cc705c2c4aa06c33361b11c06423ff76d9eed7c

Simplifications to sqlite3BtreeInsert() and allocateSpace().  Added many
testcase() macros to verify boundary conditions in btree.c. (CVS 6858)

FossilOrigin-Name: aab82a229a984bdd37bda2d140cf4279ab54a741

Improvements to corrupt database detection in defragmentPage(). (CVS 6857)

FossilOrigin-Name: 87bbc8d6b68c089c8211c35c11c2f6ac4e46271c

Do not load the root-page of a b-tree table/index when opening a cursor. Instead, allow it to be loaded when the cursor is first used (in function moveToRoot()). Also move the root-page flags sanity checks that were a part of the OP_OpenRead/OpenWrite opcodes into the moveToRoot() function. (CVS 6856)

FossilOrigin-Name: 06dcfe72a6ff3f63639eeb00ec5b5022d10fc55b

If an error occurs in PagerBegin(), call pager_end_transaction() to reset the internal state of the pager object. (CVS 6855)

FossilOrigin-Name: ea7ed16628da29cf4a127a25cbac8ae553e2ac1f

Fix a reference count problem following database corruption detection in
the ptrmapPut() routine of btree.c. (CVS 6854)

FossilOrigin-Name: ec1e27e33b89e809a03e4140b351f6c8674c0233

Simplifications to btree.c and vdbeaux.c in support of structural coverage
testing. (CVS 6853)

FossilOrigin-Name: fc2a40a9db23285da16f40f5131613907fb80f02

Fix the pager so that it remembers the name of the journal for an in-memory
database as an empty string.  This is important when committing a multi-database
transaction that includes one or more attached memory databases. (CVS 6852)

FossilOrigin-Name: 31a5e8192e164f01c5d7c2744b67cfb517251fad

Fix the backup logic so that it detects a failure to sqlite3BtreeUpdateMeta()
and reports that failure back up to the top level. (CVS 6851)

FossilOrigin-Name: 471b92f2514aedecb5773825a7ae472190375727

Make the sqlite3BtreeMoveto function static, since it is only used from within btree.c. Remove unused function lockBtreeWithRetry from btree.c. (CVS 6850)

FossilOrigin-Name: 30d5ec62ab6a85ee60ee4128e20959842f8c7ad1

Simplifications and comment cleanup in vdbeaux.c. (CVS 6849)

FossilOrigin-Name: 1636e7831a21d401a48aa74d884444a287f14f72

Remove unreachable code from function btreeCursor() in btree.c. (CVS 6848)

FossilOrigin-Name: c76a366ed4dc63604ff695b3ee9c183e430a367e

Remove a redundant branch from btree.c. (CVS 6847)

FossilOrigin-Name: 133357d2f070ba303deddff59beead1ec8d10521

Remove two unused fields from the Parse structure.  The TCL interface now
removes leading whitespace from SQL statements before parsing them. (CVS 6846)

FossilOrigin-Name: 983cb6924b8a7d3057718b9228c0cb2fbe7f0dc4

Remove unused code (that was commented out) from prepare.c.  Remove an
incorrect NEVER() macro.  Fix the comment on schemaIsValid() to correctly
describe its return behavior. (CVS 6845)

FossilOrigin-Name: 90deb65835febc908f21bca89ef497d1048afe0a

Cause sqlite3_exec() to return SQLITE_NOMEM if it cannot allocate memory to
hold the error message for some other error. (CVS 6844)

FossilOrigin-Name: e9849292ab741f4c86203b2629dbe116f9b527c8

Remove an unused variable from the test code in test3.c. (CVS 6843)

FossilOrigin-Name: 9e35bec9ef924ec44e87294a549238e0104bb27b

Add a version of the Lemon parser template "lempar.c" to the src/ subfolder.
This new parser template contains a couple of NEVER() macros that disable
tests that are needed for general grammars but not for the specific grammar
used by SQLite.  SQLite builds with the modified lempar.c. (CVS 6842)

FossilOrigin-Name: d426cc64f6014e2c2b9bcde0f5440396aec2fa18

Simplify the way the read-uncommitted flag is handled to facilitate test coverage. (CVS 6841)

FossilOrigin-Name: e2112d6160a82ccd4b6a1c798d2064d0f1b086eb

Make explicit the restrictions on UPDATE, DELETE, and INSERT statement syntax
within triggers.  Ticket #3947. (CVS 6840)

FossilOrigin-Name: c8bf40df7be728b11bb633516d1988c6064a9d70

Test the result of pthread_create() and do not call pthread_join() if the
thread creation failed.  Ticket #3933. (CVS 6839)

FossilOrigin-Name: 304c5110ad958b2cc1ddff30e68c8791109128b5

Fix to sqlite3AuthRead to accommodate "new" or "old" references that are used in a context where a column reference may also be used (i.e. "SELECT new.<col> FROM <tble>"). Ticket #3944. (CVS 6838)

FossilOrigin-Name: 45fd5419a7cde29eb6ab5d98141bd642af0d78fb

When a b-tree transaction is committed when there are open cursors, downgrade shared-cache write-locks to read-locks instead of relinquishing all locks. Fix for #3942. (CVS 6837)

FossilOrigin-Name: 611e704fdf90a3d3932ca1cbab4be7e282bf1ddf

Cause opening a transaction on a sharable b-tree module automatically obtain a read-lock on page 1. This means there is no way for sqlite3BtreeGetMeta() to fail. (CVS 6836)

FossilOrigin-Name: e3c055f167f895ae45858de9d9d8a264df2f36b6

Fix for compilation with OMIT_AUTOVACUUM defined. Ticket #3940. (CVS 6835)

FossilOrigin-Name: 8f0591ae98c2125a4922933496f4412aee8ab86e

Change to test code and scripts allowing memsys3 to be tested using a build that has both memsys3 and memsys5 enabled. (CVS 6834)

FossilOrigin-Name: a00aeec21cca7ccf19d4cdef618149aa2773c005

Fix a 1-byte buffer overwrite that can occur when a virtual table overloads an SQL function. (CVS 6833)

FossilOrigin-Name: bfe336a8993e06e70d3c32a929202fb015dcbb89

Report an error if a USING or ON clause is specified following a from-list element that is not to the right of a join operator. Fix for #3846. (CVS 6832)

FossilOrigin-Name: 29b48972b65a17dab343b063a620cf8d456a923c

Do not ignore alias "a" in a query of the form "SELECT ... FROM (...) AS a" Fix for #3935. Also expand upon (6751) to fix some similar obscure memory leaks. (CVS 6831)

FossilOrigin-Name: 42f9d1e56483a59353bff57d75f09ed67e1d9c3c

Cause incremental-blob read/write operations lock shared-cache tables in the same way as normal SQL read/writes. Add complex assert statements to make sure tehe correct shared-cache locks are held when accessing the database. Eliminate some redundant checks from btree.c. (CVS 6830)

FossilOrigin-Name: f17ef37897da9bcaf20b5acdce6840522c0a0b16

Version 3.6.16 (CVS 6829)

FossilOrigin-Name: ff691a6b2a302fe7978459cb8df9d56184892ee0

Fix an instance where sqlite3JumpHere() might be called with a negative
address following an OOM fault. (CVS 6828)

FossilOrigin-Name: 49f22e55d69d0b5a34400b36332a2eb861362eb2

Fix a bug in sqlite3_realloc() - if called with a size of more than
2147483392 it returns 0 but it also releases the prior allocation. (CVS 6827)

FossilOrigin-Name: 653df0afcc58de82c8c1b5f6a7b2f4829ff69792

Remove incorrect NEVER() macros from malloc.c.  The allocations can be
exceeded using sqlite3_malloc() and sqlite3_realloc(). (CVS 6826)

FossilOrigin-Name: 0d345e5923ff92a87195f6c04a29a56bf67ee43c

Removed a few compiler warnings under MSVC. (CVS 6825)

FossilOrigin-Name: f01a9fc375d77c67602a9f6be6a674beb516233f

Remove the priorNewRowid field from the sqlite3 structure.  Use the
last_insert_rowid as the initial value when searching for a new random
rowid in the OP_NewRowid opcode. (CVS 6824)

FossilOrigin-Name: 96729b2d499f029bfaec6648a592e8ec697d9521

Skip a couple tests requiring working 64bit int support on platforms without that support.  Test code change only. (CVS 6823)

FossilOrigin-Name: d8c8145c9d28d014eb9218b3f1312b957ccc72b7

When sqlite3VdbeAddOp() fails due to an OOM fault, return an address of 1
rather than zero so that there appears to always be a prior instruction.
This prevents a negative jump-to address computation in where.c following
an OOM fault. (CVS 6822)

FossilOrigin-Name: fe02339cb2f61c1e8a356e2d0ce9c3aef6e6cd8d

Fix a test script problem in exclusive.test. (CVS 6821)

FossilOrigin-Name: a52ef5a4216f26c617bd1471cd19f3da2a6d2c4c

Fix the new pager.c changes so that they compile with IOTRACE defined.
Fix an out-of-order variable definition in vdbeaux.c. (CVS 6820)

FossilOrigin-Name: ac1450285025e33fad81e2fb14a06eb85e8ed87a

Another change to test_journal.c to account for (6817). Again, only test code has changed. (CVS 6819)

FossilOrigin-Name: 58884b6c50f927c5606d857b2865d788a5147060

Update test_journal.c to account for (6817). Changes to test code only. (CVS 6818)

FossilOrigin-Name: 542ee8cced2a37095808d8baf5002dc66f4a64d6

Avoid writing the 8-byte journal-header magic until the journal-header is synced. In persistent journal-mode, this prevents any old content that follows an unsynced journal-header from being interpreted as part of the rollback journal. (CVS 6817)

FossilOrigin-Name: a5ecffcf025da2fcb241e83c7bebc1095a3b51d6

Fix a problem with a return code being ignored in insertCell(). (CVS 6816)

FossilOrigin-Name: bb5f1c01435dcc1ea056f0d149f750fa7812f652

Get the notify2.test running again by removing an ALWAYS() that was mistakenly
inserted and by taking care not to report corruption following an
SQLITE_LOCKED failure during schema parsing. (CVS 6815)

FossilOrigin-Name: 6e9140a729bd26dd102e603abf6ea16d67e8546e

Replace an "if" condition that is always true in btree.c with an assert(). (CVS 6814)

FossilOrigin-Name: daadbd9a8f0e35dd16c715cf2e9f99f52afa30dd

Improved handling of oversize string and blob errors.  Other simplifications
in support of full coverage testing. (CVS 6813)

FossilOrigin-Name: 8b34076668f0f712de0fbbe9bc2e68d42797e1b2

Remove an ALWAYS() macro from vdbe.c that sometimes fails on thread tests. (CVS 6812)

FossilOrigin-Name: 52eac078c7080519911599e19819cbec475de6d6

Disable the new malloc-37 test when locking_mode==exclusive.  The test
basically works, but sometimes reports "database is locked" instead of
"out of memory" and that messes up the error reporting. (CVS 6811)

FossilOrigin-Name: 3b8f9dec24d06865455a75587bbfc199c19c6215

Remove a declaration of the sqlite3Assert() function that was deleted by the
previous check-in. (CVS 6810)

FossilOrigin-Name: ee20b6a23fa4a4ec4d3a2d0dcfcabff466f08246

Remove the sqlite3Assert() function.  The ALWAYS() and NEVER() macros call
assert() directly when compiled with SQLITE_DEBUG. (CVS 6809)

FossilOrigin-Name: d8fc373fef22311e1c6b5bce6d3e601217a69940

Simplify things by rolling the functionality of balance_shallower() into balance_nonroot(). (CVS 6808)

FossilOrigin-Name: 11750c6aee6aa05b2627ad9dfb2fbcdfe8944168

Enhance autoincrement so that it works with triggers that also do
autoincrement inserts, even multiple inserts into the same table.
Ticket #3928 (CVS 6807)

FossilOrigin-Name: 1330993de8eae7baeec24100216158063c9bdc19

Remove a condition from balance_nonroot() that is always true. (CVS 6806)

FossilOrigin-Name: c5dc80e6bdd18a5ada728c8d5c9403ac233f1c9a

Call balance_shallower() from balance_nonroot() instead of from balance(). This simplifies coverage testing a bit. (CVS 6805)

FossilOrigin-Name: da9893e23caf89090c8b6563cb5f88d7dbf7c260

Update the configure script for version 3.6.16 (CVS 6804)

FossilOrigin-Name: b614e554f7ebaef9cb1718b617e020e532568bbb

Update the version number in preparation for the next release. (CVS 6803)

FossilOrigin-Name: 207c4a02ba4dd950948367c91b4a73209d62da41

Simplifications to vdbe.c to promote better test coverage. (CVS 6802)

FossilOrigin-Name: 3ffc93d762b64fd84f47c4b6d68ab56b69ea98a9

Add a test to verify that #3929 is fixed. (CVS 6801)

FossilOrigin-Name: 16c1ae9bde895d39c4eaababbd1dbf9e3b9cb653

Fix a failing assert in btree.c. The same bug was causing a spurious SQLITE_CORRUPT return when compiled without SQLITE_DEBUG. (CVS 6800)

FossilOrigin-Name: 47ec8749470af7cab9f3ef15effce1a7ba79a654

Simplifications to vdbe.c and it service routines in support of coverage
testing. (CVS 6799)

FossilOrigin-Name: 308f2e61520ac7440700d93ca5bab4a844f2dc17

Remove a condition that is always true from btree.c. (CVS 6798)

FossilOrigin-Name: 3ec8b37a89fdb2436d312439715414fae2cd20a8

Make sure sqlite3BtreeCreateTable() returns an error code if the largest
rootpage field is invalid in auto_vacuum mode.  Update file-format comments
to describe all of the "meta" values. (CVS 6797)

FossilOrigin-Name: bda38f5e86db45234c2dbb0f8d1a81f6ff6d04e4

More simplifications to vdbe.c.  Remove a NEVER() from vdbe.c that is possible
after all. (CVS 6796)

FossilOrigin-Name: c8f009bd24cb4e4b5ce9fbfd8ab09921f19d41af

If an OOM error occurs just after obtaining a shared lock on the database file, release the lock before returning. (CVS 6795)

FossilOrigin-Name: 971a9650f66f079630489c34e40003eee97d1683

Simplifications to vdbe.c in support of coverage testing. (CVS 6794)

FossilOrigin-Name: 16680f05bd169dfb1b4bf0eb082e69701f9b07ab

When recovering from the error-state in pagerSharedLock(), do not search for a hot-journal in the file-system if the pager is configured to use an anonymous temp file as the journal. (CVS 6793)

FossilOrigin-Name: 4b46805cbc9fe71f1febe9ea5f0f5634b65128e3

If after obtaining an exclusive lock to rollback a hot-journal file it is found that the journal file has been deleted, do not return an SQLITE_BUSY error. Just downgrade the lock and continue with the current operation. This eliminates a spurious SQLITE_BUSY error caused by a race condition. (CVS 6792)

FossilOrigin-Name: 9a0666003764774b0bf861687002f8db9fd314b1