Skip to content

Commit 684d0f5

Browse files
author
Vladimir Kotal
authored
properly create temporary directory (#2264)
1 parent 87c193f commit 684d0f5

File tree

1 file changed

+2
-12
lines changed

1 file changed

+2
-12
lines changed

opengrok-indexer/src/main/java/org/opengrok/indexer/history/SSCMRepository.java

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@
3030
import java.io.IOException;
3131
import java.io.InputStream;
3232
import java.io.Reader;
33+
import java.nio.file.Files;
3334
import java.util.ArrayList;
3435
import java.util.List;
3536
import java.util.Properties;
@@ -175,20 +176,9 @@ InputStream getHistoryGet(String parent, final String basename, String rev) {
175176

176177
Process process = null;
177178
try {
178-
final File tmp = File.createTempFile("opengrok", "tmp");
179+
final File tmp = Files.createTempDirectory("opengrokSSCMtmp").toFile();
179180
String tmpName = tmp.getCanonicalPath();
180181

181-
// cleartool can't get to a previously existing file
182-
if (tmp.exists() && !tmp.delete()) {
183-
LOGGER.log(Level.WARNING,
184-
"Failed to remove temporary file used by history cache");
185-
}
186-
187-
if (!tmp.mkdir()) {
188-
LOGGER.log(Level.WARNING,
189-
"Failed to create temporary directory used by history cache");
190-
}
191-
192182
List<String> argv = new ArrayList<>();
193183
ensureCommand(CMD_PROPERTY_KEY, CMD_FALLBACK);
194184
argv.add(RepoCommand);

0 commit comments

Comments
 (0)