]> git.sur5r.net Git - openldap/log
openldap
11 years agoMore fixes for prev commit
Howard Chu [Mon, 19 Aug 2013 20:54:17 +0000 (13:54 -0700)]
More fixes for prev commit

11 years agoFix 34f832faee9f215dfdb61de52506f2905258b147
Howard Chu [Mon, 19 Aug 2013 11:20:35 +0000 (04:20 -0700)]
Fix 34f832faee9f215dfdb61de52506f2905258b147

Lock ordering issues

11 years agoFix prev commit
Howard Chu [Mon, 19 Aug 2013 04:15:30 +0000 (21:15 -0700)]
Fix prev commit

Hardcoded number of queues was leftover from preliminary
work. Use connection_pool_queues.

11 years agoMultiple queues per threadpool
Howard Chu [Thu, 15 Aug 2013 13:54:35 +0000 (06:54 -0700)]
Multiple queues per threadpool

11 years agoTweak mutex acquisition
Howard Chu [Sat, 17 Aug 2013 01:56:31 +0000 (18:56 -0700)]
Tweak mutex acquisition

Slightly reduce held time.

11 years agoMerge remote-tracking branch 'origin/mdb.master'
Howard Chu [Fri, 16 Aug 2013 13:44:58 +0000 (06:44 -0700)]
Merge remote-tracking branch 'origin/mdb.master'

11 years agoSilence warnings
Howard Chu [Wed, 14 Aug 2013 16:57:25 +0000 (09:57 -0700)]
Silence warnings

11 years agoTweak ITS#7615 fix: Obey strict-aliasing rules.
Hallvard Furuseth [Wed, 14 Aug 2013 13:21:59 +0000 (15:21 +0200)]
Tweak ITS#7615 fix: Obey strict-aliasing rules.

Do not access a long long as an array of longs.

11 years agoMore for 907b9d487753631371dd969b551b63c41cef30fd
Howard Chu [Wed, 14 Aug 2013 10:51:29 +0000 (03:51 -0700)]
More for  907b9d487753631371dd969b551b63c41cef30fd

op.o_extra is only used with threading

11 years agoMerge remote-tracking branch 'origin/mdb.master'
Howard Chu [Tue, 13 Aug 2013 20:35:46 +0000 (13:35 -0700)]
Merge remote-tracking branch 'origin/mdb.master'

11 years agoITS#7664 better fix
Howard Chu [Tue, 13 Aug 2013 20:12:47 +0000 (13:12 -0700)]
ITS#7664 better fix

For RDONLY, don't get lockfile until we know datafile exists.
Also, don't try to create a new datafile for me_mfd if someone
deleted it after we got me_fd.

11 years agoRevert "ITS#7664 delete lockfile if there is no environment"
Howard Chu [Tue, 13 Aug 2013 20:05:32 +0000 (13:05 -0700)]
Revert "ITS#7664 delete lockfile if there is no environment"

This reverts commit 8725838409512e999d78ea7b60773ce2f94d588f.

11 years agoMerge remote-tracking branch 'origin/mdb.master'
Howard Chu [Tue, 13 Aug 2013 18:36:50 +0000 (11:36 -0700)]
Merge remote-tracking branch 'origin/mdb.master'

11 years agoITS#7664 delete lockfile if there is no environment
Howard Chu [Tue, 13 Aug 2013 18:34:32 +0000 (11:34 -0700)]
ITS#7664 delete lockfile if there is no environment

11 years agoFix prev commit
Howard Chu [Mon, 12 Aug 2013 22:04:51 +0000 (15:04 -0700)]
Fix prev commit

11 years agoFixes for threaded indexer
Howard Chu [Mon, 12 Aug 2013 21:49:41 +0000 (14:49 -0700)]
Fixes for threaded indexer

Works now, but still 2x slower. Leave disabled.

11 years agoMerge remote-tracking branch 'origin/mdb.master'
Howard Chu [Mon, 12 Aug 2013 19:49:24 +0000 (12:49 -0700)]
Merge remote-tracking branch 'origin/mdb.master'

11 years agoFix uninit'd mem ref in mdb_page_split
Howard Chu [Mon, 12 Aug 2013 19:43:42 +0000 (12:43 -0700)]
Fix uninit'd mem ref in mdb_page_split

Don't compare cursors beyond their depth. (detected by valgrind)

11 years agoMerge remote-tracking branch 'origin/mdb.master'
Howard Chu [Mon, 12 Aug 2013 00:27:17 +0000 (17:27 -0700)]
Merge remote-tracking branch 'origin/mdb.master'

11 years agoTweak NODUPDATA flag check
Howard Chu [Mon, 12 Aug 2013 00:25:00 +0000 (17:25 -0700)]
Tweak NODUPDATA flag check

Should allow for the possibility of future flag combos.

11 years agoMerge remote-tracking branch 'origin/mdb.master'
Howard Chu [Mon, 12 Aug 2013 00:18:20 +0000 (17:18 -0700)]
Merge remote-tracking branch 'origin/mdb.master'

11 years agoFix obscure MDB_MULTIPLE bug
Howard Chu [Mon, 12 Aug 2013 00:15:03 +0000 (17:15 -0700)]
Fix obscure MDB_MULTIPLE bug

If a key has a single existing value, and then a put (MDB_MULTIPLE)
is done where the first of the multiple values matches the existing
value, the put would return SUCCESS without writing any of the
values. Fixed to loop to the next value as intended.

11 years agoMerge remote-tracking branch 'origin/mdb.master'
Howard Chu [Sat, 10 Aug 2013 23:20:47 +0000 (16:20 -0700)]
Merge remote-tracking branch 'origin/mdb.master'

11 years agoPoint to mdb.c for docs on macros in Makefile
Howard Chu [Sat, 10 Aug 2013 21:55:51 +0000 (14:55 -0700)]
Point to mdb.c for docs on macros in Makefile

11 years agoMerge remote-tracking branch 'origin/mdb.master'
Howard Chu [Sat, 10 Aug 2013 12:05:57 +0000 (05:05 -0700)]
Merge remote-tracking branch 'origin/mdb.master'

11 years agoITS#7662 cleanup reader txn in mdb_entry_get()
Howard Chu [Sat, 10 Aug 2013 12:05:16 +0000 (05:05 -0700)]
ITS#7662 cleanup reader txn in mdb_entry_get()

11 years agoAvoid compiling debug args with C90 compilers.
Hallvard Furuseth [Sat, 10 Aug 2013 11:50:10 +0000 (13:50 +0200)]
Avoid compiling debug args with C90 compilers.

Use double DPRINTF() parenthesis, so varargs macros are not needed.
Parenthesize MDB_DEBUG, in case the user defines it as foo || bar.

11 years agoRefuse ops on bad txns
Hallvard Furuseth [Sat, 10 Aug 2013 11:23:30 +0000 (13:23 +0200)]
Refuse ops on bad txns

11 years agoReplace EINVALs with generalized MDB_INCOMPATIBLE.
Hallvard Furuseth [Sat, 10 Aug 2013 11:23:30 +0000 (13:23 +0200)]
Replace EINVALs with generalized MDB_INCOMPATIBLE.

Also check xcursor instead of DUPSORT, it's slightly briefer.

11 years agoITS#7656 fix install target
John Hewson [Sat, 10 Aug 2013 11:12:42 +0000 (04:12 -0700)]
ITS#7656 fix install target

11 years agoITS#7660 Add mdb_txn_env to API
Salvador Ortiz [Fri, 9 Aug 2013 16:03:28 +0000 (11:03 -0500)]
ITS#7660 Add mdb_txn_env to API

11 years agomdb_txn_commit(): Always commit if MDB_TXN_SPILLS.
Hallvard Furuseth [Sat, 10 Aug 2013 10:29:45 +0000 (12:29 +0200)]
mdb_txn_commit(): Always commit if MDB_TXN_SPILLS.

Checking dirty_list was insufficient after a spill
with no named databases and no positioned cursors.

11 years agoITS#7661 mdb_dbi_flags(): Allow main DBI
Salvador Ortiz [Sat, 10 Aug 2013 07:42:19 +0000 (09:42 +0200)]
ITS#7661 mdb_dbi_flags(): Allow main DBI

11 years agoITS#7659 add directoryString compatible syntaxes
Howard Chu [Fri, 9 Aug 2013 17:08:09 +0000 (10:08 -0700)]
ITS#7659 add directoryString compatible syntaxes

countryString and telephoneNumber were missing

11 years agoMerge remote-tracking branch 'origin/mdb.master'
Howard Chu [Fri, 9 Aug 2013 16:07:14 +0000 (09:07 -0700)]
Merge remote-tracking branch 'origin/mdb.master'

11 years agomdb_del must ignore data if db not opened with MDB_DUPSORT
Salvador Ortiz [Fri, 9 Aug 2013 15:08:53 +0000 (10:08 -0500)]
mdb_del must ignore data if db not opened with MDB_DUPSORT

11 years agoMerge remote-tracking branch 'origin/mdb.master'
Howard Chu [Fri, 9 Aug 2013 15:46:47 +0000 (08:46 -0700)]
Merge remote-tracking branch 'origin/mdb.master'

11 years agoAdd mdb_env_get_maxkeysize()
Howard Chu [Fri, 9 Aug 2013 11:51:33 +0000 (04:51 -0700)]
Add mdb_env_get_maxkeysize()

11 years agoReplace unpredictable EINVAL error returns.
Hallvard Furuseth [Fri, 9 Aug 2013 11:05:14 +0000 (13:05 +0200)]
Replace unpredictable EINVAL error returns.

Return EINVAL only for simple programmer errors.

11 years agoRe-fix reader-pid code
Hallvard Furuseth [Fri, 9 Aug 2013 10:54:42 +0000 (12:54 +0200)]
Re-fix reader-pid code

11 years agomdb_cursorpages_mark: Mark current txn and no more.
Hallvard Furuseth [Thu, 8 Aug 2013 17:57:52 +0000 (19:57 +0200)]
mdb_cursorpages_mark: Mark current txn and no more.

Ignore parent txn cursors since it is the current txn's dirty_list
which will be flushed. But check the current txn also when clearing,
since cursors can have pages which are dirty in a parent.

Check !mc_xcursor instead of !MDB_DUPSORT.  Equivalent for valid
data, but a bit safer if the sub-DB flags are corrupt.

11 years agoFix mdb_ovpage_free() vs. spill.
Hallvard Furuseth [Thu, 8 Aug 2013 17:57:52 +0000 (19:57 +0200)]
Fix mdb_ovpage_free() vs. spill.

Ensure me_pghead has room before removing from spill/dirty list.
Don't return pages to me_pghead in nested txns, use mt_free_pgs.

11 years agoFix page spilling when MDB_WRITEMAP.
Hallvard Furuseth [Thu, 8 Aug 2013 17:57:51 +0000 (19:57 +0200)]
Fix page spilling when MDB_WRITEMAP.

mdb_page_spill(): Don't binary-search the unsorted dirty_list.
mdb_page_flush(): Don't overwrite unprocessed dirty_list items.

11 years agoSet MDB_TXN_ERROR when inconsistent txn state
Hallvard Furuseth [Thu, 8 Aug 2013 17:57:51 +0000 (19:57 +0200)]
Set MDB_TXN_ERROR when inconsistent txn state

11 years agoFactor out MDB_env.
Hallvard Furuseth [Thu, 8 Aug 2013 17:54:54 +0000 (19:54 +0200)]
Factor out MDB_env.

11 years agoMDB_LOCK_VERSION -> MDB_LOCK_FORMAT.
Hallvard Furuseth [Thu, 8 Aug 2013 17:43:04 +0000 (19:43 +0200)]
MDB_LOCK_VERSION -> MDB_LOCK_FORMAT.

Pid locking needs a different lockfile-version: MDB_env's with and
without pid locking must not coexist, they can sabotage each other.
Store MDB_LOCK_FORMAT = (version | "use locking" flag) instead.

11 years agoFix mdb_reader_pid().
Hallvard Furuseth [Thu, 8 Aug 2013 17:43:04 +0000 (19:43 +0200)]
Fix mdb_reader_pid().

Treat unexpected errors as "don't know".  Invert Pidcheck return
value, so nonzero including error codes = "the process may exist".

On Windows: Catch exited but still existing processes.  Handle
undefined PROCESS_QUERY_LIMITED_INFORMATION.
On Unix: don't trust F_GETLK error to leave the input alone,
the fcntl() doc seems unclear.

11 years agoUse proper printf format on Windows
Howard Chu [Wed, 7 Aug 2013 19:42:46 +0000 (12:42 -0700)]
Use proper printf format on Windows

11 years agoITS#7645 more for TLSProtocolMin
Howard Chu [Wed, 7 Aug 2013 18:08:09 +0000 (11:08 -0700)]
ITS#7645 more for TLSProtocolMin

11 years agoSilence warnings
Hallvard Furuseth [Mon, 5 Aug 2013 08:01:39 +0000 (10:01 +0200)]
Silence warnings

11 years agoTweak comments
Hallvard Furuseth [Mon, 5 Aug 2013 07:55:57 +0000 (09:55 +0200)]
Tweak comments

11 years agoClarify doc: mdb_copy, nested txns, mdb_drop().
Hallvard Furuseth [Mon, 5 Aug 2013 07:55:48 +0000 (09:55 +0200)]
Clarify doc: mdb_copy, nested txns, mdb_drop().

mdb_copy: Does not copy lockfile. Can trigger file growth.
mdb_txn_begin(): Clarify usage restrictions.
mdb_drop(): State what to do rather than what will be done, since
closing the handle could otherwise be read as happening even at failure.

11 years agoITS#7591 fix back-meta missing result msgs
Howard Chu [Fri, 2 Aug 2013 11:44:57 +0000 (04:44 -0700)]
ITS#7591 fix back-meta missing result msgs

11 years agoITS#7654 fix implicit declaration warnings
Howard Chu [Fri, 2 Aug 2013 11:03:05 +0000 (04:03 -0700)]
ITS#7654 fix implicit declaration warnings

11 years agoMerge remote-tracking branch 'origin/mdb.master'
Howard Chu [Wed, 31 Jul 2013 15:14:39 +0000 (08:14 -0700)]
Merge remote-tracking branch 'origin/mdb.master'

11 years agoTweak prev commit again
Howard Chu [Wed, 31 Jul 2013 15:09:40 +0000 (08:09 -0700)]
Tweak prev commit again

Make sure errors are propagated from init_meta

11 years agoMerge remote-tracking branch 'origin/mdb.master'
Howard Chu [Tue, 30 Jul 2013 20:48:01 +0000 (13:48 -0700)]
Merge remote-tracking branch 'origin/mdb.master'

11 years agoFix typo in Win32 branch
Howard Chu [Tue, 30 Jul 2013 20:44:28 +0000 (13:44 -0700)]
Fix typo in Win32 branch

11 years agoMerge remote-tracking branch 'origin/mdb.master'
Howard Chu [Tue, 30 Jul 2013 19:51:54 +0000 (12:51 -0700)]
Merge remote-tracking branch 'origin/mdb.master'

11 years agoCleanup prev commit
Howard Chu [Tue, 30 Jul 2013 19:47:12 +0000 (12:47 -0700)]
Cleanup prev commit

Loop on copyfd meta write, since pipes may return after partial write.

11 years agoITS#7652 fix I/O error checks
Howard Chu [Tue, 30 Jul 2013 17:22:12 +0000 (10:22 -0700)]
ITS#7652 fix I/O error checks

partial revert of d6d2638acc245116b8f091ac425b6700d06c4713 and
26a25df5fcc2fcddae6597a61c1b867fc27c568b

The original code was already tested and working correctly.

11 years agoITS#7645 use ldap_pvt_tls_config for TLSProtocolMin
Howard Chu [Mon, 29 Jul 2013 14:04:03 +0000 (07:04 -0700)]
ITS#7645 use ldap_pvt_tls_config for TLSProtocolMin

11 years agoITS#7645 more OpenSSL TLS versions
Philip Guenther [Mon, 29 Jul 2013 14:01:15 +0000 (07:01 -0700)]
ITS#7645 more OpenSSL TLS versions

11 years agoITS#7645, #5655 TLSProtocolMin docs
Howard Chu [Mon, 29 Jul 2013 13:50:18 +0000 (06:50 -0700)]
ITS#7645, #5655 TLSProtocolMin docs

11 years agoITS#5655 cleanup
Howard Chu [Mon, 29 Jul 2013 13:40:02 +0000 (06:40 -0700)]
ITS#5655 cleanup

11 years agoITS#5655 TLS_PROTOCOL_MIN setting
Philip Guenther [Mon, 29 Jul 2013 13:38:27 +0000 (06:38 -0700)]
ITS#5655 TLS_PROTOCOL_MIN setting

11 years agoMerge remote-tracking branch 'origin/mdb.master'
Howard Chu [Mon, 29 Jul 2013 00:13:04 +0000 (17:13 -0700)]
Merge remote-tracking branch 'origin/mdb.master'

11 years agoTweak mdb_envinfo numreaders
Howard Chu [Mon, 29 Jul 2013 00:02:51 +0000 (17:02 -0700)]
Tweak mdb_envinfo numreaders

Return the actual shared reader count when it exists, not
just the current process env's reader count.

11 years agoITS#7641 don't muck with controls if slapi doesn't use them
Timothy Pearson [Fri, 26 Jul 2013 20:15:58 +0000 (13:15 -0700)]
ITS#7641 don't muck with controls if slapi doesn't use them

11 years agoITS#7615 use shorter names for semaphores
Howard Chu [Fri, 26 Jul 2013 17:19:54 +0000 (10:19 -0700)]
ITS#7615 use shorter names for semaphores

NetBSD can only handle up to 14 chars, we were using 21. Now
we encode to 15, and for NetBSD truncate the last char.

11 years agoMerge remote-tracking branch 'origin/mdb.master'
Howard Chu [Fri, 19 Jul 2013 17:02:24 +0000 (10:02 -0700)]
Merge remote-tracking branch 'origin/mdb.master'

11 years agoTweak reader_pid check
Howard Chu [Fri, 19 Jul 2013 16:57:33 +0000 (09:57 -0700)]
Tweak reader_pid check

Check again after acquiring rmutex. Avoids potential issue with
a duplicate pid coming in between initial check and rmutex.

11 years agoGet pid lock outside of rmutex
Howard Chu [Fri, 19 Jul 2013 16:55:10 +0000 (09:55 -0700)]
Get pid lock outside of rmutex

Avoid holding rmutex for longer than necessary.

11 years agoMerge remote-tracking branch 'origin/mdb.master'
Howard Chu [Thu, 18 Jul 2013 22:25:29 +0000 (15:25 -0700)]
Merge remote-tracking branch 'origin/mdb.master'

11 years agoTweak reader checks
Howard Chu [Thu, 18 Jul 2013 22:24:09 +0000 (15:24 -0700)]
Tweak reader checks

Use mti_numreaders for loop limit, not me_maxreaders.

11 years agoMerge remote-tracking branch 'origin/mdb.master'
Howard Chu [Thu, 18 Jul 2013 18:09:05 +0000 (11:09 -0700)]
Merge remote-tracking branch 'origin/mdb.master'

11 years agoAdd mdb_reader_check()
Howard Chu [Thu, 18 Jul 2013 17:40:21 +0000 (10:40 -0700)]
Add mdb_reader_check()

11 years agoSplit MDB_VERSION to MDB_DATA/MDB_LOCK VERSION
Howard Chu [Thu, 18 Jul 2013 16:11:09 +0000 (09:11 -0700)]
Split MDB_VERSION to MDB_DATA/MDB_LOCK VERSION

11 years agoTweak reader_list
Howard Chu [Thu, 18 Jul 2013 16:00:51 +0000 (09:00 -0700)]
Tweak reader_list

11 years agoTweak mdb_stat(1)
Howard Chu [Thu, 18 Jul 2013 15:33:24 +0000 (08:33 -0700)]
Tweak mdb_stat(1)

Don't obtain reader txn before displaying reader table. Exit
after reader table if no other DB query options were given.

11 years agoAdd mdb_reader_list()
Howard Chu [Thu, 18 Jul 2013 14:41:11 +0000 (07:41 -0700)]
Add mdb_reader_list()

Dump the active slots in the reader table.

11 years agoAdd mdb_dbi_flags()
Howard Chu [Mon, 15 Jul 2013 17:57:13 +0000 (10:57 -0700)]
Add mdb_dbi_flags()

Retrieve the flags from a DB handle.

11 years agoMerge remote-tracking branch 'origin/mdb.master'
Howard Chu [Mon, 15 Jul 2013 00:20:14 +0000 (17:20 -0700)]
Merge remote-tracking branch 'origin/mdb.master'

11 years agoFix child txn dirty_room counts in spill/unspill
Howard Chu [Sun, 14 Jul 2013 23:53:04 +0000 (16:53 -0700)]
Fix child txn dirty_room counts in spill/unspill

Don't count pages twice if they're already accounted in an ancestor txn.

11 years agoMore for stale sub-cursor flags
Howard Chu [Sun, 14 Jul 2013 15:28:26 +0000 (08:28 -0700)]
More for stale sub-cursor flags

Same fix for cursor_first/last.

11 years agoFix stale sub-cursor C_INIT flag
Howard Chu [Sun, 14 Jul 2013 15:20:18 +0000 (08:20 -0700)]
Fix stale sub-cursor C_INIT flag

Whenever we enter cursor_set() the sub-cursor's flag must be
cleared. If the new cursor position has valid subdata it will
be initialized again, if not then the sub-cursor has nothing
to point to.

11 years agoTweak comments, defaults should be OK already
Howard Chu [Fri, 12 Jul 2013 20:55:18 +0000 (13:55 -0700)]
Tweak comments, defaults should be OK already

11 years agoBump version to 0.9.7
Howard Chu [Fri, 12 Jul 2013 20:36:05 +0000 (13:36 -0700)]
Bump version to 0.9.7

11 years agoMerge remote-tracking branch 'origin/mdb.master'
Howard Chu [Fri, 12 Jul 2013 20:23:45 +0000 (13:23 -0700)]
Merge remote-tracking branch 'origin/mdb.master'

11 years agoAlso set/clear P_KEEP in parent txn's cursors
Hallvard Furuseth [Fri, 12 Jul 2013 09:30:33 +0000 (11:30 +0200)]
Also set/clear P_KEEP in parent txn's cursors

11 years agoSpill pages, take 3
Howard Chu [Thu, 11 Jul 2013 20:09:47 +0000 (22:09 +0200)]
Spill pages, take 3

11 years agoDelay touching pages until cursor is positioned.
Howard Chu [Thu, 11 Jul 2013 20:09:47 +0000 (22:09 +0200)]
Delay touching pages until cursor is positioned.

This avoids unnecessary rewrites of pages that do not change.
(Restructuring for upcoming mdb_page_spill work.)

11 years agoSimplify: Always set C_UNTRACK for tracked cursors.
Hallvard Furuseth [Thu, 11 Jul 2013 20:09:46 +0000 (22:09 +0200)]
Simplify: Always set C_UNTRACK for tracked cursors.

TODO: Rename C_UNTRACK to C_TRACKED.  Omitted now for readability.
The current name is because it's lazy: not always set when tracked.

11 years agoSave freelist using proper mdb_cursor_put().
Hallvard Furuseth [Thu, 11 Jul 2013 20:09:46 +0000 (22:09 +0200)]
Save freelist using proper mdb_cursor_put().

(Restructuring for upcoming mdb_page_spill work.)
mdb_freelist_save() can't just Get() the destination, since
mdb_page_spill() may have put the destination in the read-only map.

TODO: Can this new put() modify the freelist, which would break it? The
final iteration's put() can shorten the node, the rest uses MDB_CURRENT.

We could set P_KEEP on dirty freeDB leaves and ovpages, since they are
all about to be modified.  But the code in this commit must stay anyway,
if mdb should support dropping a 256G DB. I.e. too big for dirty_list.

11 years agoMove code out to mdb_page_dirty()
Howard Chu [Thu, 11 Jul 2013 20:09:46 +0000 (22:09 +0200)]
Move code out to mdb_page_dirty()

11 years agoFactor out parent
Howard Chu [Thu, 11 Jul 2013 20:09:46 +0000 (22:09 +0200)]
Factor out parent

11 years agoFix env_read_header() on Windows
Howard Chu [Fri, 12 Jul 2013 19:53:35 +0000 (12:53 -0700)]
Fix env_read_header() on Windows

Commit d6d2638acc245116b8f091ac425b6700d06c4713 broke read
on zero-length files.

11 years agoDo not follow uninited cursors' page pointers.
Hallvard Furuseth [Wed, 10 Jul 2013 20:11:44 +0000 (22:11 +0200)]
Do not follow uninited cursors' page pointers.

Nor uninited cursors' subcursors' page pointers.

11 years agoITS#7636 slapi: don't free filterstr if filter didn't change
Howard Chu [Wed, 10 Jul 2013 19:56:04 +0000 (12:56 -0700)]
ITS#7636 slapi: don't free filterstr if filter didn't change

11 years agoFix rebalance/cursor adjust
Howard Chu [Wed, 10 Jul 2013 18:03:51 +0000 (11:03 -0700)]
Fix rebalance/cursor adjust

When collapsing root, must also move cursor index down,
not just the page pointer.

Also in mtest, break from NEXT loops on error, otherwise it just
prints the previous key/data again, which looks confusing.