ITS#6758 fix/add slap_send_search_entry() comments

ITS#6758 result.c:Use rs_*(). Frees entries earlier.

For readability, use rs_flush_entry() where slap_send_search_entry() and
slap_send_search_reference() release entries before network operations.

This change also frees entires marked MUSTBEFREED earlier.

ITS#6758 Use rs_*() to manage SlapReply entries.
Some noop changes + fixes entry leaks and REP_ENTRY_MODIFIABLE flag leaks.

noop commit: cleanup 0 -> REP_RESULT in SlapReply

ITS#6758, pcache.c: Avoid SlapReply reuse.
refresh_merge(): Do not reuse Search SlapReply parameter for a Modify,
use a local one and copy its sr_err to the parameter.  The parameter is
from refresh_query().  Move its SlapReply parameter to a local variable.

ITS#6758 Avoid SlapReply reuse.
Caller provides a SlapReply and reads the result after the operation,
then reuses it.  Reinitialize before reuse, or move declaration inwards.

ITS#6760: Fix rwm entry handling

ITS#6758 Avoid SlapReply reuse, simple cases.
Caller provides and reuses a SlapReply it does not read after
initialization.  So reinitialize or move the declaration inwards.
In case of refint, move the rs parameter into refint_repair().

ITS#6758 rs_reinit() += slap_reply_t parameter

ITS#6758 config_build_entry, avoid SlapReply reuse

Do not use the 'SlapReply *rs' passed to config_build_entry().  Instead
pass a new one to op->o_bd->be_add() and copy the output sr_err to rs.
This takes care of most SlapReply reuse in bconfig.c.

ITS#6758 bconfig.c: ensure sr_text==NULL if sr_err==LDAP_SUCCESS
config_modify_internal(): Do not send rs->sr_text with success.
config_back_modify(): When slap_bv2ad() can set rs->sr_text,
set sr->sr_err too and catch failure.

ITS#6670 partial fix - ignore member/memberof pointing to same entry

ITS#6758 back-bdb: Delay setting sr->sr_entry until we know it will be used

ITS#6758 Do not set sr_err=error before be_search.
glue_op_search() set LDAP_UNWILLING_TO_PERFORM before be_search() as
well as when there was no be_search.

ITS#6758 Reset some SlapReply flags & data.
Reset data more consistenlty.  When resetting sr_entry after sending
data, reset sr_flag as well.  Reset rs_attrs and Extended Operation
oid/data after setting and sending them.

noop commit for ITS#6758: simplify future commits

noop commit for ITS#6758: Remove no-effect code.
Unused variable resets, unused SlapReply parameters to some functions.

Remove lint (unused vars/labels/funcs/default:s)

ITS#6758 Add rs_reinit(), rs_flush_entry()

ITS#6758 Add rs_assert*, RS_ASSERT, USE_RS_ASSERT

ITS#6759: assert;Debug -> Debug;assert

silence valgrind

should also work for SASL binds (whenever supported)

op->o_conn->c_sb may be 0 for internal operations

ITS#6754: Fail if there are unrecognized command-line arguments.

ITS#6741 support Bitstring in ldap_X509dn2bv()

ITS#6684 fix ppolicy crash, fix Makefile install target

Remove dead code.

ITS#5862 Assert control for rootDSE/subschema

s/0/O/

fix perror; removed extra sequence opening (is it needed; if yes, it must also be closed, see #if 0)

verify credentials extop: support for simple bind

s/VCRequest/VCResponse/

Fix prototype to match code

ITS#6686 Fix last commit (declaration after code)

ITS#6742 from Rich Megginson @ Red Hat

ITS#6453 cleanup on LDAP_TIMEOUT

ITS#6547 latest from FSF http://savannah.gnu.org/git/?group=config

ITS#6686 add multiple sorts per connection, from Raphael Ouazana @ Linagora

ITS#6681 fix examples. Hopefully admins are smart enough to insert
the correct uid themselves.

Move assertion checking to slap_compare_entry()

ITS#6753 use slap_compare_entry()

ITS#6573 export compare_entry as slap_compare_entry()

Cleanup for #6660

ITS#6660 preliminary fix. debug additions need to be cleaned up.

Plug one-time LDAPMessage leak

Plug mutex/rwlock leaks (destroy them)

Plug memory leaks

ITS#6752 Remove OpenLDAP 2.[23] dynlist code

ITS#6752 Handle REP_ENTRY_* flags correctly

ITS#6752 Fix dynlist Compare callback
Replace dynlist_sc_save_entry() with dynlist_sc_compare_entry() which
compares instead of saves the entry.  Check rs->sr_type == REP_SEARCH.

ITS#6752 dynlist: Do less work, simplify a bit.
Rearrange dynlist_compare(), tweak dynlist_response().

Add basic client

Misc vc updates

Add comment regarding ASN.1 syntax
rm version field

Fix typo

Very back ldapvc (verify credentials) client

should be .5

Assign experimental OID

cleanup

LDAP "Verify Credentials" operation

Client library implementation only

fix perror

Use ldap_parse_whoami

ITS#6748 Drop possibly-broken memberof log message

ITS#6747 Fix LDAP_CONNECTIONLESS Debug(), warnings

ITS#6746: Preserve error code, plug entry leak

Reduced debug level, the default logs more than 1Gb.

ITS#6716: Keep sessionlog sorted by csn, compare mincsn with head.

ITS#6714 non-blocking support from Richard Megginson @ Red Hat

Cleanup prev commit

ITS#6744 - plug leak

Partial fix for non-string types in cert DNs. (Need to add explicit support
for Bitstring, maybe a few others.)

re-fix ITS#6645 commit

fix ITS#6645 commit

move ctxcsn and schema check code in helpers; also apply to slapmodify (ITS#6737)

allow to set LDIF max line width (ITS#6645)

typo in comment

ITS#6625 Remove some LDAP_R_COMPILEs

Fix ITS#6625: mutex-protect ldap_free_connection()

ITS#6625 Invert test in NT threads LDAP_INT_MUTEX_FIRSTCREATE

ITS#6625 concurrency patch cleanup

For ITS#6739: Fix last commit (private debuging)

For ITS#6739: Move variables into do_syncrep2 loop

Silence valgrind's ptrcheck (no real change).
Also removes a bit unnecessary code.

ITS#6736: Delay destruction of slap_listeners[]

ITS#6735 Fix uninitialized filter in slapd-tester

Fix rev 1.270: free def_urlpre correctly.  Also free urlpre.

ITS#6733: Fix LDAP_MEMORY_DEBUG issues.
ber_memcalloc_x overflow check.  Silence some compiler warnings.

Fix rev 1.132 (ITS#6632): Initialize all of roc in rwm_callback_get()

ITS#6732: Clean up ber_errno handling.
Set it after ber_mem(c)alloc_x failure.  Don't set it when not needed.

ITS#6620: Don't delete absent pwdChangedTime.
Based on patch by Martin Evans.

ITS#6731: Fix scrambled back-ldif error strings.
* Move apply_modify_to_entry():textbuf[] into caller.
* Reset rs->sr_text at end of functions that can set it to textbuf.

ITS#6364: Document that refint needs rootdn

ITS#6730: Only update modifiersName once per entry

ITS#6730: Fix missing refint updates with subtrees/subordinates

ITS#6663: Fix refint_nothing.
Also move callback trace output out of inner loop in callback.

test slap_dontUseCopy_propnames (ITS#6729)

fix double free related to ITS#6720

Fix a comment

Add/fix comments.  No other change.