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
Don't smash passwords with newly alloc'd zero array
Merge pull request #402 from olivierbeytrison/fix-policies

check for attribute existance in policies
Remove from the proxy list if we can't insert it
Only print foreach close brace once.  Closes #377
free -> talloc_free
Actually use virtual server when proxying to it.  Closes #375
Fix horrific spacing issue.
Correct call to paircreate in radeapclient
Merge pull request #405 from olivierbeytrison/fix-radeapclient

Correct call to paircreate in radeapclient
Update stats structure (remove separate loss structure)
Better checks for drops
Fix email in copyright
Move default secret to radsniff.h
Freeing the filter tree should be one of the last things we do...