[accumulo] branch master updated: Fix spotbugs sec-bugs

[ct...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

ctubbsii pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/accumulo.git


The following commit(s) were added to refs/heads/master by this push:
     new 31b5424  Fix spotbugs sec-bugs
31b5424 is described below

commit 31b54248176f320cc15f432ece29f998a0d3a363
Author: Christopher Tubbs <ct...@apache.org>
AuthorDate: Mon Feb 11 18:18:24 2019 -0500

    Fix spotbugs sec-bugs
---
 .travis.yml                                                   | 2 +-
 test/src/main/java/org/apache/accumulo/test/DumpConfigIT.java | 4 ++++
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/.travis.yml b/.travis.yml
index 484a54a..ee9f99b 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -27,7 +27,7 @@ before_script:
   - unset _JAVA_OPTIONS
 env:
   - BUILD_CMD="mvn clean verify javadoc:jar -DskipITs"  # main build of unit tests and javadoc
-  - BUILD_CMD="travis_wait 20 mvn clean verify -PskipQA,sec-bugs -Dspotbugs.skip=false -Dspotbugs.timeout=1200000"  # check for spotbugs+security
+  - BUILD_CMD="travis_wait 30 mvn clean verify -PskipQA,sec-bugs -Dspotbugs.skip=false -Dspotbugs.timeout=1800000"  # check for spotbugs+security
   - BUILD_CMD="mvn clean compile -Dhadoop.version=3.0.3 -Dzookeeper.version=3.4.12"  # quick compile to verify older API
 script:
   - $BUILD_CMD
diff --git a/test/src/main/java/org/apache/accumulo/test/DumpConfigIT.java b/test/src/main/java/org/apache/accumulo/test/DumpConfigIT.java
index 5f2a859..c36552a 100644
--- a/test/src/main/java/org/apache/accumulo/test/DumpConfigIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/DumpConfigIT.java
@@ -34,6 +34,8 @@ import org.apache.hadoop.conf.Configuration;
 import org.junit.Test;
 import org.junit.rules.TemporaryFolder;
 
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
+
 public class DumpConfigIT extends ConfigurableMacBase {
 
   @Override
@@ -46,6 +48,8 @@ public class DumpConfigIT extends ConfigurableMacBase {
     cfg.setSiteConfig(Collections.singletonMap(Property.TABLE_FILE_BLOCK_SIZE.getKey(), "1234567"));
   }
 
+  @SuppressFBWarnings(value = "PATH_TRAVERSAL_IN",
+      justification = "user.dir is suitable test input")
   @Test
   public void test() throws Exception {
     File target = new File(System.getProperty("user.dir"), "target");