Remove double assignment (its assigned after the ns validation)
Remove unused (and broken) function

Can result in a null pointer dereference as it
doesn't check the return value from bson_next()
Make static analyzers happy.. cursor_id and int64_id will never both be set

But thats not something a static analyzer understands
Make sure we have a command
Since we know the string length, use it
No need to reset something that wasn't set
Fix a memory leak when one of the servers is blacklisted
Merge pull request #537 from bjori/fix-test-memleaks

Fix compiler and build-scan warnings

* bjori/fix-test-memleaks:
  Fix a memory leak when one of the servers is blacklisted
  No need to reset something that wasn't set
  Since we know the string length, use it
  Make sure we have a command
  Make static analyzers happy.. cursor_id and int64_id will never both be set
  Remove unused (and broken) function
  Remove double assignment (its assigned after the ns validation)