* Use mkstemp() to create the temporary credential file instead of
  using a filename based upon timestamps.  We had tens or hundreds of
  munge auth failures per day when using the timestamp-based temporary
  credential file names due to the fact that they were not unique.

* Make daughter routines of req_altauthenuser() return PBSE_* codes.
  Replace req_reject() calls in these routines with log_err() calls
  using the aforementioned PBSE codes.  Add log_err() calls in these
  routines for additional error conditions for complete coverage, so
  we can actually tell from the server log what exactly went wrong if
  a munge auth attempt failed.  Call req_reject() once in
  req_altauthenuser() instead.

* Use non-blocking I/O for reading the munge credential in
  pipe_and_read_unmunge().

* Don't delete the credential file if there was a problem unmunge-ing
  the request.
Merge pull request #236 from craigprescott/4.2-dev

Munge auth: credential filename uniqueness, improved error logging