mir/libreoffice
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

svx: fix another temp file leak, from SdrMediaObj's glTF models

Browse Source 
These need a whole directory, apparently a model may have multiple
files.

Change-Id: I092c1764ddb1ed30947034f299fbb6882658d21b
This commit is contained in:
Michael Stahl
2014-10-14 15:04:26 +02:00
parent 616445cde0
commit 782caa59cc
1 changed files with 19 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

27
svx/source/svdraw/svdomedia.cxx
View File

@@ -41,6 +41,7 @@
// For handling of glTF models // For handling of glTF models
#include <unotools/tempfile.hxx> #include <unotools/tempfile.hxx>
#include <unotools/localfilehelper.hxx>
#include <tools/urlobj.hxx> #include <tools/urlobj.hxx>
using namespace ::com::sun::star; using namespace ::com::sun::star;
@@ -54,10 +55,17 @@ using namespace ::com::sun::star;
struct MediaTempFile struct MediaTempFile
{ {
OUString const m_TempFileURL; OUString const m_TempFileURL;
MediaTempFile(OUString const& rURL) : m_TempFileURL(rURL) {} OUString const m_TempDirURL; // yet another hack, for the glTF models
MediaTempFile(OUString const& rURL, OUString const& rDirURL)
: m_TempFileURL(rURL), m_TempDirURL(rDirURL)
{}
~MediaTempFile() ~MediaTempFile()
{ {
::osl::File::remove(m_TempFileURL); ::osl::File::remove(m_TempFileURL);
if (!m_TempDirURL.isEmpty())
{
::utl::removeTree(m_TempDirURL);
}
} }
}; };
@@ -270,10 +278,11 @@ uno::Reference<io::XInputStream> SdrMediaObj::GetInputStream()
static bool lcl_HandleJsonPackageURL( static bool lcl_HandleJsonPackageURL(
const OUString& rURL, const OUString& rURL,
SdrModel* const pModel, SdrModel* const pModel,
OUString& o_rTempFileURL) OUString& o_rTempFileURL,
OUString& o_rTempDirURL)
{ {
// Create a temporary folder which will contain all files of glTF model // Create a temporary folder which will contain all files of glTF model
const OUString sTempFolder = ::utl::TempFile( NULL, true ).GetURL(); o_rTempDirURL = ::utl::TempFile(NULL, true).GetURL();
const sal_uInt16 nPackageLength = OString("vnd.sun.star.Package:").getLength(); const sal_uInt16 nPackageLength = OString("vnd.sun.star.Package:").getLength();
const OUString sUrlPath = rURL.copy(nPackageLength,rURL.lastIndexOf("/")-nPackageLength); const OUString sUrlPath = rURL.copy(nPackageLength,rURL.lastIndexOf("/")-nPackageLength);
@@ -298,7 +307,7 @@ static bool lcl_HandleJsonPackageURL(
{ {
// Generate temp file path // Generate temp file path
const OUString& rFilename = aFilenames[nFileIndex]; const OUString& rFilename = aFilenames[nFileIndex];
INetURLObject aUrlObj(sTempFolder); INetURLObject aUrlObj(o_rTempDirURL);
aUrlObj.insertName(rFilename); aUrlObj.insertName(rFilename);
const OUString sFilepath = aUrlObj.GetMainURL( INetURLObject::NO_DECODE ); const OUString sFilepath = aUrlObj.GetMainURL( INetURLObject::NO_DECODE );
@@ -367,7 +376,7 @@ void SdrMediaObj::SetInputStream(uno::Reference<io::XInputStream> const& xStream
bool const bSuccess = lcl_CopyToTempFile(xStream, tempFileURL); bool const bSuccess = lcl_CopyToTempFile(xStream, tempFileURL);
if (bSuccess) if (bSuccess)
{ {
m_pImpl->m_pTempFile.reset(new MediaTempFile(tempFileURL)); m_pImpl->m_pTempFile.reset(new MediaTempFile(tempFileURL, ""));
m_pImpl->m_MediaProperties.setURL( m_pImpl->m_MediaProperties.setURL(
m_pImpl->m_LastFailedPkgURL, tempFileURL, ""); m_pImpl->m_LastFailedPkgURL, tempFileURL, "");
} }
@@ -429,16 +438,18 @@ void SdrMediaObj::mediaPropertiesChanged( const ::avmedia::MediaItem& rNewProper
rNewProperties.getTempURL())) rNewProperties.getTempURL()))
{ {
OUString tempFileURL; OUString tempFileURL;
OUString tempDirURL;
bool bSuccess; bool bSuccess;
#if HAVE_FEATURE_GLTF #if HAVE_FEATURE_GLTF
if( url.endsWith(".json") ) if( url.endsWith(".json") )
bSuccess = lcl_HandleJsonPackageURL(url, GetModel(), tempFileURL); bSuccess = lcl_HandleJsonPackageURL(url, GetModel(), tempFileURL, tempDirURL);
else else
#endif #endif
bSuccess = lcl_HandlePackageURL( url, GetModel(), tempFileURL); bSuccess = lcl_HandlePackageURL(url, GetModel(), tempFileURL);
if (bSuccess) if (bSuccess)
{ {
m_pImpl->m_pTempFile.reset(new MediaTempFile(tempFileURL)); m_pImpl->m_pTempFile.reset(
new MediaTempFile(tempFileURL, tempDirURL));
m_pImpl->m_MediaProperties.setURL(url, tempFileURL, ""); m_pImpl->m_MediaProperties.setURL(url, tempFileURL, "");
} }
else // this case is for Clone via operator= else // this case is for Clone via operator=