Add \0 safe parsing of LDAP binary attributes
Add some more asserts for catching invalid VPs when build without WITH_VERIFY_PTR
pairparsevalue should return 0 or -1 like pretty much every other function int the server
Don't need memset if it's static
Need extra break
Status should always be initialised in rlm_ldap_bind

We'll never actually hit this (it would require fr_connection_get_num to return a negative value).
Don't set nonblock on outgoing client connections.

The caller will need to set it if necessary
Fix use of fr_nonblock && listen() for sockets

Outgoing TCP sockets are non-blocking.  Incoming sockets
are non-blocking.  Outgoing TLS sockets are blocking.

Don't call listen() on outgoing sockets
certs will be NULL for outgoing proxy sockets.
Add the proxy listener to the packet list immediately

so that we can allocate IDs immediately
only use TLS if TLS is enabled
close_notify isn't an error.  It's a polite notification
Don't print out useless proxy ID
It's OK to send packets in outgoing proxy socket INIT state
Formatting
Comment
Remove code in radius_map2vp for !* ANY (it's not needed)
Fix double & with attribute references in debug_map

It's completely pointless, so much so there's not
Print maps for !* ANY
Add macro for FR_FAULT_LOG