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

Fail early if getPackageLocation fails; plus clean-up

Browse Source 
Change-Id: Ifa7d589e7854fed9cd9ca2a3f3c987179b60199f
This commit is contained in:
Stephan Bergmann
2012-10-01 11:46:24 +02:00
parent 585ad1cde0
commit a900a4d70b
1 changed files with 15 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files

33
sdext/source/pdfimport/wrapper/wrapper.cxx
View File

@@ -44,7 +44,7 @@
#include "com/sun/star/io/XInputStream.hpp"
#include "com/sun/star/uno/XComponentContext.hpp"
#include "com/sun/star/awt/FontDescriptor.hpp"
#include "com/sun/star/deployment/XPackageInformationProvider.hpp"
#include "com/sun/star/deployment/PackageInformationProvider.hpp"
#include "com/sun/star/beans/XMaterialHolder.hpp"
#include "com/sun/star/rendering/PathCapType.hpp"
#include "com/sun/star/rendering/PathJoinType.hpp"
@@ -999,24 +999,21 @@ bool xpdf_ImportFromFile( const ::rtl::OUString& rUR
if( checkEncryption( aSysUPath, xIHdl, aPwd, bIsEncrypted, aDocName ) == false )
return false;
rtl::OUStringBuffer converterURL = rtl::OUString("xpdfimport");
// retrieve package location url (xpdfimport executable is located there)
// ---------------------------------------------------
uno::Reference<deployment::XPackageInformationProvider> xProvider(
xContext->getValueByName(
rtl::OUString("/singletons/com.sun.star.deployment.PackageInformationProvider")),
uno::UNO_QUERY);
if( xProvider.is() )
{
converterURL.insert(
0,
rtl::OUString("/"));
converterURL.insert(
0,
xProvider->getPackageLocation(
rtl::OUString::createFromAscii(PDFI_IMPL_IDENTIFIER)));
// Retrieve package location URL, xpdfimport executable is located there:
OUString location(
deployment::PackageInformationProvider::get(xContext)->
getPackageLocation(PDFI_IMPL_IDENTIFIER));
if (location.isEmpty()) {
SAL_WARN(
"sdext.pdfimport",
"getPackageLocation(" PDFI_IMPL_IDENTIFIER ") failed");
return false;
}
rtl::OUStringBuffer converterURL(location);
if (!location.endsWith("/")) {
converterURL.append('/');
}
converterURL.append("xpdfimport");
// spawn separate process to keep LGPL/GPL code apart.
// ---------------------------------------------------