Fix obvious pasto in PostOrder walk.
Add rbtree_callbydata for sane threadsafe/garbage-collected operations

  When working with an rbtree which is exposed to multiple threads,
  it is not safe to do much of anything with data retrieved by functions
  such as rbtree_finddata, other than to feed it directly to rbtree_delete.

  This is because once rbtree_finddata has returned, another thread may
  obtain a pointer to that data (and hence may begin mangling non-key
  material such as container_of or any child allocs.)  This is especially
  true for rbtrees that have a freeNode garbage collection routine defined.

  This function allows safe operations while the rbtree lock is still
  held.  It also allows for a conditional delete operation based on
  criteria which may only be safe to ascertain while the lock is held.
  In addition to short operations, it could be used, with due care, to
  trylock a more granular lock associated with the key before deleting
  the key or before operating with it outside of the rbtree lock.
Use a consistent callback return code

  This is the one place in the codebase where a positive return code
  is used in an rbtree_walk callback.  Changing it negative should have no
  effect, and keeps options open as to whether to introduce new symbols
  for the next patch to follow, or to change the libfreeradius API
  slightly WRT rbtree callback return codes.

  Whether any third-party add-ons may use positive return codes is not
  known.
Always fixup after deleting a black child node in rbtree

	This prevents rbtree_delete from creating unbalanced/illegal rbtrees.
Add a batch deletion mode to rbtree_walk

   Walks the tree calling callbacks as per InOrder
   Allows callback to ask for deletion of records after evaluating criteria
   All callbacks and deletions done while lock is held
A test file for libradius rbtree ops
    Verifies red/black counts to all child nodes after adds/deletes
    Tests DeleteOrder; code provided in comments for vanilla rbtree_delete.
    Build system magic necessary for this is beyond me.
    It's a start at least
When a socket is closed, just mark the requests as STOP

Someone else will take care of cleaning them up.  Calling
request_done() appears to cause issues
Fix obvious pasto in PostOrder walk.
Use a consistent callback return code

  This is the one place in the codebase where a positive return code
  is used in an rbtree_walk callback.  Changing it negative should have no
  effect, and keeps options open as to whether to introduce new symbols
  for the next patch to follow, or to change the libfreeradius API
  slightly WRT rbtree callback return codes.

  Whether any third-party add-ons may use positive return codes is not
  known.
Always fixup after deleting a black child node in rbtree

	This prevents rbtree_delete from creating unbalanced/illegal rbtrees.
Non-existent attrs expand to "".  Closes #308
Don't try and link if legacy files exist already. Fixes #341
Don't start if the configuration contains deprecated config items
Apparently making the symlink targets as order only dependencies isnt good enough
Update README.rst
Update README.rst
If anything exists don't create the link...
loc is not options
suse: build fixes

* Merge most changes to suse/freeradius.spec from v2.x.x branch.
* Add suse/freeradius-server-rpmlintrc and
  suse/freeradius-server-tmpfiles.conf from v2.x.x branch.
* Merge changes on suse/freeradius-server-2.1.6-suseinit.patch.
* Add BuildRequires: libtalloc-devel.
* Remove BuildRequires: apache2-devel. Dialup admin was removed.
* Remove BuildRequires: cyrus-sasl-devel, openssl. Already installed
  as requirement for other BuildRequires.
* Remove BuildRequires: gettext-devel, libtool, net-snmp-devel.
  No longer needed.
* Remove unneeded configure flags.
* Make main package more generic by using wildcard in %files. For
  example, it will put resulting oracle libs (if enabled during build)
  on the main package.
May as well make it official