moparisthebest/poi
1
0
Fork 0
Code Issues Pull Requests Releases 1 Wiki Activity

[bug-62176] synchronize temp dir creation

Browse Source 
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1826655 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
PJ Fanning 2018-03-13 15:12:00 +00:00
parent fd39985247
commit 47fb3691f7
2 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

BIN
gradle/wrapper/gradle-wrapper.jar vendored
View File

Binary file not shown.

5
src/java/org/apache/poi/util/DefaultTempFileCreationStrategy.java
View File

@ -85,11 +85,12 @@ public class DefaultTempFileCreationStrategy implements TempFileCreationStrategy
/** /**
* Attempt to create a directory, including any necessary parent directories. * Attempt to create a directory, including any necessary parent directories.
* Does nothing if directory already exists. * Does nothing if directory already exists.
* The method is synchronized to ensure that multiple threads don't try to create the directory at the same time.
* *
* @param directory the directory to create * @param directory the directory to create
* @throws IOException if unable to create temporary directory or it is not a directory * @throws IOException if unable to create temporary directory or it is not a directory
*/ */
private void createTempDirectory(File directory) throws IOException { private synchronized void createTempDirectory(File directory) throws IOException {
// create directory if it doesn't exist // create directory if it doesn't exist
final boolean dirExists = (directory.exists() || directory.mkdirs()); final boolean dirExists = (directory.exists() || directory.mkdirs());
@ -138,4 +139,4 @@ public class DefaultTempFileCreationStrategy implements TempFileCreationStrategy
// All done // All done
return newDirectory; return newDirectory;
} }
} }