Don't crash when querying a collector whose name doesn't resolve. #3946

In CollectorList::query(), check for daemon->addr() being NULL before
checking if it's blacklisted, as the blacklist check asssumes addr()
isn't NULL.
Docs for bad collector name crash bug fix. #3946
Windows fix: do not pass bad params to fstat(), preventing a except. #4097

fstat() is considered a security related libc call on Windows, and when
security related calls get passed unexpected data (like a bad file
descriptor), Windows invokes a process-wide Invalid Parameter Handler
Routine.  The default handler crashes the application (on purpose).
This patch makes sure the descriptor returned by open() is valid
before handing it down to fstat().
Merge branch 'V8_0-branch' of ssh://tonic.cs.wisc.edu/p/condor/repository/CONDOR_SRC into V8_0-branch
Merge branch 'V8_0-branch'