Issue 101891 - Access rights detection does not work if file is in afs filesystem
Summary: Access rights detection does not work if file is in afs filesystem
Status: UNCONFIRMED
Alias: None
Product: General
Classification: Code
Component: code (show other issues)
Version: OOo 3.0.1
Hardware: Unknown Linux, all
: P3 Trivial with 2 votes (vote)
Target Milestone: ---
Assignee: AOO issues mailing list
QA Contact:
URL:
Keywords: needhelp
: 104922 (view as issue list)
Depends on:
Blocks:
 
Reported: 2009-05-13 17:12 UTC by angelmarin
Modified: 2014-01-22 16:06 UTC (History)
4 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---

Attachments
Patch to make OOo use access() (1.60 KB, patch)
2010-03-22 00:04 UTC, mbechler 		no flags Details | Diff
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description angelmarin 2009-05-13 17:12:03 UTC 
OpenOffice apps since 3.0 try to figure out write access permissions of the file
they're opening and open it in read only mode if it determines it won't be able
to write back to that file.

Problem is that on filesystems like afs that have their own acl system and
completely ignore the regular unix perms, OO.org apps fail to open in read/write
mode any file you didn't create.

The only current workaround is to chmod all OO.org documents as 666, and let
openafs filesystem acls enforce read/write perms.

If compatibility with afs acl system is not planned, at least not doing this
permission check when files reside in a afs filesystem would be good enough for us.

BTW, problem has been reported to exist on 3.1 too
(http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=524108)
Comment 1 mbechler 2010-03-22 00:03:21 UTC 
Still a bug in 3.2.0. The issue is that you are using the file unix mode to
determine access rights on your own. This needs to be done by access() calls
(your way also won't work in an acl scenario, which sucks too). As stated in
source the other side is

> We don't use access(...) because access follows links which
> may cause performance problems see #97133.

but please, at least make this compile-time configurable. A patch is attached,
please review.
Comment 2 mbechler 2010-03-22 00:04:09 UTC 
Created attachment 68463 [details]
Patch to make OOo use access()
Comment 3 Edwin Sharp 2014-01-22 16:05:33 UTC 
*** Issue 104922 has been marked as a duplicate of this issue. ***