OpenAFS Master Repository branch, master, updated. openafs-devel-1_5_69-15-g3f89c0f

Gerrit Code Review gerrit@openafs.org
Fri, 22 Jan 2010 10:50:16 -0800 (PST)


The following commit has been merged in the master branch:
commit 3f89c0feae89e9a255afb8a7f08995412a3f1b79
Author: Andrew Deason <adeason@sinenomine.net>
Date:   Thu Jan 21 20:42:33 2010 -0500

    Fix afs_AccessOK for dropbox case
    
    afs_AccessOK did not check for if we have 'i' rights on a directory, nor
    if we were the owner of a file, if we were only checking PRSFS_READ or
    PRSFS_WRITE. Thus, it does not correctly address the 'dropbox'
    permissions case (if you are the owner of a file and have insert
    permissions, you get implicit 'r' and 'w').
    
    Fix this to check for 'i' and ownership when we need to, so the dropbox
    special-case check works as intended.
    
    FIXES 126216
    
    Change-Id: Ib629bc30b73bc3965e777747067259451f0adff6
    Reviewed-on: http://gerrit.openafs.org/1143
    Tested-by: Andrew Deason <adeason@sinenomine.net>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>

 src/afs/VNOPS/afs_vnop_access.c |   31 ++++++++++++++++++++++++++-----
 1 files changed, 26 insertions(+), 5 deletions(-)

-- 
OpenAFS Master Repository