ITS#7577 more for previous commit

ITS#7577 alias fixes

Merge remote-tracking branch 'origin/mdb.master' into OPENLDAP_REL_ENG_2_4

Fix 66c839f0292d9aea12bf93ef6568ac2bd2ca0fc8

Forgot #include <errno.h>

Refactor mdb_midl_append, add mdb_midl_grow()

ITS#7515 Fix nested transaction error handling.

mdb_txn_begin(): Do not free(mt_free_pgs), it needs mdb_midl_free().
mdb_txn_commit(): Catch commit(child) error.

Fix mdb_env_close(unopened MDB_env).

Do not try to scan me_dbxs in a closed/never-opened MDB_env.
Broken by 7d643d3acb94a6d7306330b550fce0e5755444a9
and 151c416b460b02cd96502a47189cda19ab651ada.

Simplify/cleanup mdb_cursor_init, C_ALLOCD.

No real change.
mdb_cursor_init() checks if it needs mx, so pass it unconditionally.
Set C_ALLOCD for shadow cursors, for clarity. (It was always set as
it should anyway from the origin cursor, which would have C_ALLOCD.)

MDB warning cleanup.

Unused function when MDB_DEBUG. Unused 'excl' param.

mdb_dcmp(): Assume the database has MDB_DUPSORT.

There was little point in returning EINVAL when not: Comparing (A,B)
and (B,A) would claim (A > B && B > A), which could confuse callers.

Update MDB doc: Cursors, DB handles, data lifetime

Allow mdb_cursor_close() after readonly txn ends.

Catch mdb_cursor_renew(write txn's cursor).  Add flag C_UNTRACK, so
mdb_cursor_close need not peek inside a possibly-freed readonly txn.

Close cursors when commit(writer). Factor out code.

Close remaining cursors when committing a write txn. The doc says this
happens, and it avoids cursor tracking when updating mainDB + freeDB.

Rename mdb_cursor_merge() -> mdb_cursors_close() for code reuse,
and add a merge option.  Simplify its loop a bit.

Factor out cleanup of DBIs.

Plug txn leak and MDB_NOTLS slot leaks on error.

On mdb_env_copy() error: Abort the txn.
On mdb_txn_renew0() error: Release new MDB_NOTLS reader slot.

Cleaner "Support mdb_txn_abort(a reset txn)".

Redo 8a562f560edcc6417672df453c4e626dfb4c4024 so !mt_dbxs indicates
"txn was reset", so mt_numdbs gets one magic value instead of two.

Update fixes for dbi_open/close, ITS#7515.

Reset me_dbflags[dbi] when closing DBI, to get rid of MDB_VALID flag.
mdb_env_close(): Re-fix DB-name memleak. DBIs > me_numdbs may exist.

Allow reading freelist while working on it

The circular dependency issues appear to have been resolved.
Still, need to watch closely, maybe revert this change if
problems arise.

Merge remote-tracking branch 'origin/mdb.master' into OPENLDAP_REL_ENG_2_4

Avoid assert

Due to underfilled branch page. We're in the process of merging/moving
nodes to it because we already know it's underfilled. Took this approach
rather than just removing the assert in mdb_page_search_root, because
that assert may yet catch other situations we don't know about.

(Although, it has been there since the original commit of mdb.c and
has never triggered any other times...)

Plug mdb_cursor_renew() memleak

ITS#7513

ITS#7513 added TCP keepalive support to back-meta

Drop ldap_int_sasl_mutex

It was introduced for Cyrus 1.5 in 2001; we've been on 2.x since 2002 and
Cyrus does its own locking when needed.

ITS#7524

Don't expire a conn if it's still referenced

ITS#7524 make sure binding flag is set before trying to bind

ITS#7518 fix manpage

ITS#7518 resync nss-pam-ldapd files

Also note minimum version required.

Fix MDB_LAST, reset cursor index

Fix typo from 7aba5f5ab92a3fa92de001edb2c5ff4b04fd7d0d

Tweak comments/readability.

Show MDB_PERSISTENT/MDB_VALID/DB_VALID relationship.
mdb_txn_renew0(): Remove obsolete "cannot fail" comment.

Add MDB_NOTLS envflag.

Catch MDB txn reuse/sync errors.

Support mdb_txn_abort(a reset txn).

Cleanup for TLS key and read-only filesystem.

Move key init into mdb_env_setup_locks().
Don't create unused TLS key when read-only filesystem.
Drop internal flag MDB_ROFS, we can instead test either
!me_txns, !mt_u.reader or me_lfd==INVALID_HANDLE_VALUE.

Clean up MDB_env setup.

Malloc before I/O. Avoids possible malloc error after I/O.
Don't allocate dirty & free lists when MDB_RDONLY.
Factor out code.

Fix mdb_rebalance

Don't do anything with (fake root) subpages

ITS#7574 blind fix

Fix 227329c8e1fb818548f1936f5dc1349ffd733523, don't persist
the MDB_VALID bit in db.md_flags.

fix prev commit

Fix mdb_rebalance

Need to check NUMKEYS as well as fill threshold, when deciding
whether to change anything. Don't let the page drop below the
minimum number of keys.

Revert "ITS#7570 fix MDB_GET_MULTIPLE and MDB_NEXT_MULTIPLE"

This reverts commit 372b3ddb85bef953fb548fbf2b9ef0cba4e3c8ed.

Original code works as designed.

ITS#7570 fix MDB_GET_MULTIPLE and MDB_NEXT_MULTIPLE

ITS#7568

ITS#7568 Add -Q to slaptest's help

ITS#7568 Fix typos in ldap.conf.5

ITS#7569

ITS#7569 fixed a group enumeration bug (Symas#2255)

ITS#6365

ITS#6365 wait for read locks in tool mode

Fix commit e5b96f2c764f2ca8106560a9e46d3a9880cc2a12

ITS#7562

no need to mention replog any longer (ITS#7562)

ITS#7556,ITS#7515,ITS#7561

attrdb fixes

Merge remote-tracking branch 'origin/mdb.master' into OPENLDAP_REL_ENG_2_4

Fix 227329c8e1fb818548f1936f5dc1349ffd73352

mdb_dbi_close() must allow any dbi < maxdbs, since opens in an
uncommitted txn don't increment env->me_numdbs.

Revert "Fix 6beaad52129da5353fd40c0ec48e6a78c4f71a2e"

This reverts commit 8eef7a4275eda8f2fa2e0d1e67c1d5cbcd91607e.
Previous commit was correct, duplicate keys should not appear here.

Fixes for dbi_open/close, ITS#7515

ITS#7561 Fix mdb_drop bugs

Check for overflow pages, reset cursor position properly.

Fix 6beaad52129da5353fd40c0ec48e6a78c4f71a2e

In APPEND don't immediately reject matching key, since this
is valid for APPENDDUP.

Fix d3990eb2f8b2944782604a2ca7bd52e08c58fc3e

Append mode should *reject* keys that are too small.
Also allow APPENDDUP in mdb_put().

Fix rebalance when collapsing root page

When fixing other cursors, must also fix their depth.

More fixes for MDB_LAST

Make sure C_INITIALIZED gets set on successful call

ITS#7556 fix cursor_sibling for PREV

Return to RE

Plug MDB memory leak of DB names.

Silence signedness warning

Fix MDB_DEBUG compile when no varargs macros.

Variables mdb_debug, mdb_debug_start were used undefined.

Tweak mode_t def per Hallvard's suggestion

MSVC doesn't define mode_t

Fix CURSOR_LAST/CURSOR_PREV

Prep for release

ITS#7553

Merge remote branch 'origin/mdb.master' into OPENLDAP_REL_ENG_2_4

ITS#7553 better fix

ITS#7552

ITS#7552 fix for templates with * / +

ITS#7553 deinit cursor after deleting last node

Silence stupid warnings

ITS#7522

ITS#7522 fix config syntax errors

Fix bdb/hdb in example

ITS#7546 fix example DNs

ITS#7551

Merge remote branch 'origin/mdb.master' into OPENLDAP_REL_ENG_2_4

ITS#7534

fix tool name (ITS#7534)

more about ITS#7525

bugfix for mdb_cursor_put with MDB_MULTIPLE

If the variable dkey.mv_size is non-zero, then it means dkey
contains some original data which has to be put back in the
child db, typically when the child db has just been created.

But when using MDB_MULTIPLE, if this variable has not been
reset to zero, we may come back to this section and wrongly
think that there is some original data to be taken care of.

ITS#7540

ITS#7540 reject undefined attrs

ITS#7543

ITS#7543 Fix log output "conn=-1 ... ACCEPT"

ITS#7542, ITS#7432

ITS#7542, more for ITS#7432

typos in quicksort

ITS#7531

Avoid unnecessary syncCookie malloc/free

ITS#7531 check updateCookie status for INTERMEDIATEs

Was already being checked for other message types.

ITS#7533

ITS#7533 don't install DB_CONFIG if no BDB backends

ITS#7536, ITS#7377