[jboss-jira] [JBoss JIRA] (ELY-1263) Coverity, Dereference null return value (PeriodicRotatingFileAuditEndpoint)

[Yeray Borges (JIRA)]

     [ https://issues.jboss.org/browse/ELY-1263?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Yeray Borges reassigned ELY-1263:
---------------------------------

    Assignee: Yeray Borges  (was: Darran Lofthouse)


> Coverity, Dereference null return value (PeriodicRotatingFileAuditEndpoint)
> ---------------------------------------------------------------------------
>
>                 Key: ELY-1263
>                 URL: https://issues.jboss.org/browse/ELY-1263
>             Project: WildFly Elytron
>          Issue Type: Bug
>            Reporter: Martin Choma
>            Assignee: Yeray Borges
>
> Coverity found potential dereference of null object , because {{file.getParentFile()}} can return null in some circumstances.
>  
> {code:java|title=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);
>         }
>     }
> {code}
> https://scan7.coverity.com/reports.htm#v23632/p11778/fileInstanceId=25046999&defectInstanceId=5660941&mergedDefectId=1445725



--
This message was sent by Atlassian JIRA
(v7.2.3#72005)