Related: tdf#88314 delete temp files

clear to close the inputstream before attempting to delete the temp file, which
is plausible as the windows-specific failure in earlier attempts at this

Change-Id: I751aa3a098dd960f56b77a5f5fc93783c1060556
Reviewed-on: https://gerrit.libreoffice.org/17531Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

package/inc/ZipOutputEntry.hxx

@@ -69,6 +69,7 @@ public:
     css::uno::Reference< css::io::XInputStream > getData() const;
     ::css::uno::Any getParallelDeflateException() const { return m_aParallelDeflateException; }
     void closeBufferFile();
+    void deleteBufferFile();
 
     ZipEntry* getZipEntry() { return m_pCurrentEntry; }
     ZipPackageStream* getZipPackageStream() { return m_pCurrentStream; }

package/source/zipapi/ZipOutputEntry.cxx

@@ -114,6 +114,13 @@ void ZipOutputEntry::closeBufferFile()
     m_xOutStream.clear();
 }
 
+void ZipOutputEntry::deleteBufferFile()
+{
+    assert(!m_xOutStream.is() && !m_aTempURL.isEmpty());
+    uno::Reference < ucb::XSimpleFileAccess3 > xAccess(ucb::SimpleFileAccess::create(m_xContext));
+    xAccess->kill(m_aTempURL);
+}
+
 uno::Reference< io::XInputStream > ZipOutputEntry::getData() const
 {
     uno::Reference < ucb::XSimpleFileAccess3 > xTempAccess(ucb::SimpleFileAccess::create(m_xContext));

package/source/zipapi/ZipOutputStream.cxx

@@ -121,10 +121,12 @@ void ZipOutputStream::finish()
             rawWrite(aSequence);
         }
         while (nRead == n_ConstBufferSize);
+        xInput.clear();
 
         rawCloseEntry(m_aEntries[i]->isEncrypt());
 
         m_aEntries[i]->getZipPackageStream()->successfullyWritten(m_aEntries[i]->getZipEntry());
+        m_aEntries[i]->deleteBufferFile();
         delete m_aEntries[i];
     }