Uploaded image for project: 'WildFly Elytron'
  1. WildFly Elytron
  2. ELY-1593

Coverity static analysis, Resource leak on the exceptional path of setFile in FileAuditEndpoint

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Won't Do
    • Icon: Major Major
    • None
    • 1.3.2.Final
    • Audit
    • None

      Coverity static analysis found possible occurrence of resource leak.

      https://scan7.coverity.com/reports.htm#v20225/p11778/fileInstanceId=53592668&defectInstanceId=11119606&mergedDefectId=1429222

      When fos.getFD() will throw exception FileOutputStream fos won't be closed.

      FileAuditEndpoint.java
          void setFile(final File file) throws IOException {
              boolean ok = false;
              final FileOutputStream fos = new FileOutputStream(file, true);
              try {
                  final OutputStream bos = new BufferedOutputStream(fos);
                  try {
                      this.fileDescriptor = fos.getFD();
                      this.outputStream = bos;
                      this.file = file;
                      ok = true;
                  } finally {
                      if (! ok) {
                          safeClose(bos);
                      }
                  }
              } finally {
                  if (! ok) {
                      safeClose(fos);
                  }
              }
          }
      

              rhn-support-ivassile Ilia Vassilev
              rhn-support-ivassile Ilia Vassilev
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated:
                Resolved: