[Libreoffice-commits] core.git: sal/osl

Isamu Mogi saturday6c at gmail.com
Thu Jun 6 06:10:10 PDT 2013


 sal/osl/w32/file_url.cxx |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit 6d2e3bdac27ade56031d930c85e906c0d35877bc
Author: Isamu Mogi <saturday6c at gmail.com>
Date:   Tue Jun 4 13:10:44 2013 +0000

    fdo#41226 Add error handling of recursed GetCaseCorrectPathNameEx()
    
    GetCaseCorrectPathNameEx() with bCheckExistence = true doesn't support windows
    share path but occasionally it doesn't return failure and returns broken result.
    For example, when we call with "\\USER-PC\Users\foo" then it converts the path
    to "\Users\foo". And when "\Users\foo" exists, it returns "\Users\foo". It is
    caused by missing error handling of searching for file "\\USER-PC". Also similar
    bug possibly occurs even for local file path. This commit fixes these bugs.
    
    Change-Id: Idf1dd1282510d8a6810ed239a83051bb0f8e2f87
    Reviewed-on: https://gerrit.libreoffice.org/4151
    Reviewed-by: Luboš Luňák <l.lunak at suse.cz>
    Tested-by: Luboš Luňák <l.lunak at suse.cz>

diff --git a/sal/osl/w32/file_url.cxx b/sal/osl/w32/file_url.cxx
index fb1fa45..e9be895 100644
--- a/sal/osl/w32/file_url.cxx
+++ b/sal/osl/w32/file_url.cxx
@@ -486,7 +486,8 @@ static DWORD GetCaseCorrectPathNameEx(
             else
                 bSkipThis = FALSE;
 
-            GetCaseCorrectPathNameEx( lpszPath, cchBuffer, nSkipLevels, bCheckExistence );
+            if ( !GetCaseCorrectPathNameEx( lpszPath, cchBuffer, nSkipLevels, bCheckExistence ) )
+                return 0;
 
             PathAddBackslash( lpszPath, cchBuffer );
 


More information about the Libreoffice-commits mailing list