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

[Hank Leininger]

 desktop/unx/source/start.c |    9 +++++++--
 sal/osl/unx/pipe.cxx       |    4 ++--
 2 files changed, 9 insertions(+), 4 deletions(-)

New commits:
commit 87c011134031374cf9104ec2fc39ef121d8a6802
Author: Hank Leininger <hl...@korelogic.com>
Date:   Fri Feb 26 20:23:03 2016 -0500

    tdf#98210 do not require R_OK for pipe dir
    
    Also better error handling if pipe dirs are really not usable.
    
    Change-Id: I1c865b9a9f1f08d2cffd07343494feca585ec75e
    Reviewed-on: https://gerrit.libreoffice.org/22727
    Tested-by: Jenkins <c...@libreoffice.org>
    Reviewed-by: jan iversen <j...@documentfoundation.org>

diff --git a/desktop/unx/source/start.c b/desktop/unx/source/start.c
index 525fe22..b072f0f 100644
--- a/desktop/unx/source/start.c
+++ b/desktop/unx/source/start.c
@@ -323,10 +323,15 @@ get_pipe_path( rtl_uString *pAppPath )
     if ( !pMd5hash )
         rtl_uString_new( &pMd5hash );
 
-    if ( access( PIPEDEFAULTPATH, R_OK|W_OK ) == 0 )
+    if ( access( PIPEDEFAULTPATH, W_OK ) == 0 )
         rtl_uString_newFromAscii( &pResult, PIPEDEFAULTPATH );
-    else
+    else if ( access( PIPEALTERNATEPATH, W_OK ) == 0 )
         rtl_uString_newFromAscii( &pResult, PIPEALTERNATEPATH );
+    else
+    {
+        fprintf( stderr, "ERROR: no valid pipe path found.\n" );
+        exit( 1 );
+    }
 
     rtl_uString_newFromAscii( &pTmp, "/OSL_PIPE_" );
     rtl_uString_newConcat( &pResult, pResult, pTmp );
diff --git a/sal/osl/unx/pipe.cxx b/sal/osl/unx/pipe.cxx
index 9321395..37ac7ec 100644
--- a/sal/osl/unx/pipe.cxx
+++ b/sal/osl/unx/pipe.cxx
@@ -166,11 +166,11 @@ oslPipe SAL_CALL osl_psz_createPipe(const sal_Char 
*pszPipeName, oslPipeOptions
     bool bNameTooLong = false;
     oslPipe  pPipe;
 
-    if (access(PIPEDEFAULTPATH, R_OK|W_OK) == 0)
+    if (access(PIPEDEFAULTPATH, W_OK) == 0)
     {
         strncpy(name, PIPEDEFAULTPATH, sizeof(name));
     }
-    else if (access(PIPEALTERNATEPATH, R_OK|W_OK) == 0)
+    else if (access(PIPEALTERNATEPATH, W_OK) == 0)
     {
         strncpy(name, PIPEALTERNATEPATH, sizeof(name));
     }
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits