Coverity CID 44669, 44670

Coverity CID 44699

Coverity CID 44730

Coverity CID 44966

Coverity CID 44770

No other change to logic, but this allowed some code reuse

Coverity CID 44675

Use the partition specified in the reservation, rather than
using the cluster's default.

Bug 4030.

Already handled on line 2080.

Bug 4030.

Bug 4117

Bug 4117

Files generated and left around by another user will now cause the
  test to fail gracefully and print a descriptive message

Coverity CID 44759

No changes to logic

Coverity CID 44779

This should be redundant
Coverity CID 45168

Coverity CID 45168

Uninitialized fd value meant the open was skipped
after commit aacd6b65.

Otherwise the job step could have already ended and the WAKE RPC
  could be rejected with some misleading communication error
  being send to srun

Don't flag a job as "SPAWNED" for debugger until all process
  information is available for all pack job components.

Previous code could exit srun before all I/O was processed.

Coverity CID 44695

Coverity CID 171475

Coverity CID 171464

Coverity CID 44842

Coverity CID 44760

Coverity CID 171456

Coverity CID 44846

Swap banking and TRES talks, Change "SLURM" to "Slurm"

Adding a newline prevents this error:
conftest.c:154:8: error: if statement has empty body [-Werror,-Wempty-body]

remote cluster correctly determine the select type.

Bug 2329

Modify "srun --mpi=list" output to match valid option input by removing the
    "mpi/" prefix on each line of output.