Make sure everything has a justification.

And move a couple of items to the exclude file. If we ever get to a lower threshold, these will show up all over the place.
This commit is contained in:
Jeff Thompson committed 2020-01-23 14:55:59 -07:00
1 parent c76ee59810
commit ced25e51fd
10 files changed
+17 -12

No files matched your search

+1 -1
View File
@@ -216,7 +216,7 @@ public abstract class Proc {
* @param err
* null to redirect stderr to stdout.
*/
@SuppressFBWarnings("COMMAND_INJECTION")
@SuppressFBWarnings(value = "COMMAND_INJECTION", justification = "Command injection is the point of this old, barely used class.")
public LocalProc(String[] cmd,String[] env,InputStream in,OutputStream out,OutputStream err,File workDir) throws IOException {
this( calcName(cmd),
stderr(environment(new ProcessBuilder(cmd),env).directory(workDir), err==null || err== SELFPUMP_OUTPUT),