-
Bug
-
Resolution: Done
-
Major
-
None
-
None
-
None
Coverity found potential dereference of null object , because file.getParentFile() can return null in some circumstances.
PeriodicRotatingFileAuditEndpoint.java
@Override protected void preWrite(Instant instant) { final long recordMillis = instant.toEpochMilli(); if (recordMillis >= nextRollover) { try { final File file = getFile(); if (file == null) { // no file is set; a direct output stream or writer was specified return; } closeStreams(); // close the original file (some OSes won't let you move/rename a file that is open) final Path target = file.getParentFile().toPath().resolve(file.getName() + nextSuffix); Files.move(file.toPath(), target, StandardCopyOption.REPLACE_EXISTING); setFile(file); } catch (IOException e) { audit.unableToRotateLogFile(e); } calcNextRollover(recordMillis); } }
- clones
-
JBEAP-11774 Coverity, Dereference null return value (PeriodicRotatingFileAuditEndpoint)
- Closed