diff options
author | Jan Holesovsky <kendy@collabora.com> | 2015-07-06 15:16:56 +0200 |
---|---|---|
committer | Jan Holesovsky <kendy@collabora.com> | 2015-07-06 15:53:26 +0200 |
commit | b183507ee293d8bcafa9c1c5b2844b7a83fea17b (patch) | |
tree | 0fdfad8686b2f97b67acb732f606573fccc302ac | |
parent | 8468436bf65b05ab77c3414e6cd5d532c6aea1d6 (diff) |
LOK: Cleanup absolutizing of URLs.
Thanks to Stephan Bergmann.
Change-Id: I22aa3bb827db28bce3eabebb9b8c514663fad860
-rw-r--r-- | desktop/source/lib/init.cxx | 35 |
1 files changed, 25 insertions, 10 deletions
diff --git a/desktop/source/lib/init.cxx b/desktop/source/lib/init.cxx index 3a0ce67af246..63fe90b585b2 100644 --- a/desktop/source/lib/init.cxx +++ b/desktop/source/lib/init.cxx @@ -26,8 +26,9 @@ #include <osl/file.hxx> #include <osl/process.h> #include <osl/thread.h> -#include <rtl/strbuf.hxx> #include <rtl/bootstrap.hxx> +#include <rtl/strbuf.hxx> +#include <rtl/uri.hxx> #include <cppuhelper/bootstrap.hxx> #include <comphelper/dispatchcommand.hxx> #include <comphelper/lok.hxx> @@ -165,19 +166,21 @@ static OUString getUString(const char* pString) static OUString getAbsoluteURL(const char* pURL) { OUString aURL(getUString(pURL)); - - // return unchanged if it likely is an URL already - if (aURL.indexOf("://") > 0) + if (aURL.isEmpty()) return aURL; - OUString sAbsoluteDocUrl, sWorkingDir, sDocPathUrl; - // convert relative paths to absolute ones - osl_getProcessWorkingDir(&sWorkingDir.pData); - osl::FileBase::getFileURLFromSystemPath( aURL, sDocPathUrl ); - osl::FileBase::getAbsoluteFileURL(sWorkingDir, sDocPathUrl, sAbsoluteDocUrl); + OUString aWorkingDir; + osl_getProcessWorkingDir(&aWorkingDir.pData); + + try { + return rtl::Uri::convertRelToAbs(aWorkingDir + "/", aURL); + } + catch (const rtl::MalformedUriException &) + { + } - return sAbsoluteDocUrl; + return OUString(); } extern "C" @@ -349,6 +352,12 @@ static LibreOfficeKitDocument* lo_documentLoadWithOptions(LibreOfficeKit* pThis, SolarMutexGuard aGuard; OUString aURL(getAbsoluteURL(pURL)); + if (aURL.isEmpty()) + { + pLib->maLastExceptionMsg = "Filename to load was not provided."; + SAL_INFO("lok", "URL for load is empty"); + return NULL; + } pLib->maLastExceptionMsg.clear(); @@ -415,6 +424,12 @@ static int doc_saveAs(LibreOfficeKitDocument* pThis, const char* sUrl, const cha OUString sFormat = getUString(pFormat); OUString aURL(getAbsoluteURL(sUrl)); + if (aURL.isEmpty()) + { + gImpl->maLastExceptionMsg = "Filename to save to was not provided."; + SAL_INFO("lok", "URL for save is empty"); + return false; + } try { |