diff options
author | 2017-11-29 23:39:35 -0800 | |
---|---|---|
committer | 2017-11-29 23:41:14 -0800 | |
commit | 3766619412ed91a2d293f612ee1490fc56560fc9 (patch) | |
tree | 6baa7d577c40d1140cf4efa263afae12b410d3e2 | |
parent | 2874302e49f3e92bb455dd3ce11f67260b618746 (diff) |
Explicitly validate that the argument to JarCreator.addDirectory exists
Follow-up to https://github.com/bazelbuild/bazel/commit/0b2352de3101e87647d083f6089246079dda0f75
PiperOrigin-RevId: 177415788
-rw-r--r-- | src/java_tools/buildjar/java/com/google/devtools/build/buildjar/jarhelper/JarCreator.java | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/jarhelper/JarCreator.java b/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/jarhelper/JarCreator.java index 097f6d7bbf..bb8be68004 100644 --- a/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/jarhelper/JarCreator.java +++ b/src/java_tools/buildjar/java/com/google/devtools/build/buildjar/jarhelper/JarCreator.java @@ -109,6 +109,9 @@ public class JarCreator extends JarHelper { * @param directory the directory to add to the jar */ public void addDirectory(Path directory) { + if (!Files.exists(directory)) { + throw new IllegalArgumentException("directory does not exist: " + directory); + } try { Files.walkFileTree( directory, |