Remove unneeded checks in rlm_sql.c
Make comparison functions use a generic comparator type

Add fr_quicksort to quicksort arrays using a comparator

Fix issue in radsniff, where the list of DICT_ATTRs needed to be sorted but weren't, meaning most of them would probably have been ignored
Remove assert (it was weird) - absence doesn't trigger complaint from clang scan
Only iterate over VPs if we have a request: CID #1211052
Hack to temporarily enable dumpable flag, call panic_action, then disable it again (if required)
Parent is the special node NIL, not NULL.  CID #1187991
Don't unlink the socket.  Closes CID #720456

The pattern of unlinking before bind is so that we don't open
a socket owned by someone else.  We already check that, so
unlinking serves no purpose.

Also, in close_socket(), unlink the file if it was a domain
socket.  That way it doesn't stick around
Quiet gcc
Doxygen
Final fix for previous commit
Apparently we need unlink?
Include NDEBUG state when printing out server features
Linux is crazy
Revert changes...
Don't alloc any memory inside in fr_fault or functions called by fr_fault.