1. In UdpServer HandleReceive/SendInternal do not deallocate the
   buffer if the UdpServer state is not OK since it will already
   be freed during Shutdown. This was uncovered during coverage
   runs.
2. Fix SyslogCollectorTest crash when running multiple iterations
   by moving Sandesh::Uninit before the thread join in
   Teardown(). Ran 100 iterations without any crash.

Closes-Bug: #1373076
Change-Id: I34d663b8c817c71b61c696736a7ab6abb78d05dc
Merge "1. In UdpServer HandleReceive/SendInternal do not deallocate the    buffer if the UdpServer state is not OK since it will already    be freed during Shutdown. This was uncovered during coverage    runs. 2. Fix SyslogCollectorTest crash when running multiple iterations    by moving Sandesh::Uninit before the thread join in    Teardown(). Ran 100 iterations without any crash."