OpenAFS Master Repository branch, openafs-devel-1_7_x, updated. openafs-devel-1_7_15-53-gf49dc58

Gerrit Code Review gerrit@openafs.org
Wed, 4 Jul 2012 16:39:07 -0700 (PDT)


The following commit has been merged in the openafs-devel-1_7_x branch:
commit f49dc589ece4ea468c5c6343d19bda1ed4a07d55
Author: Jeffrey Altman <jaltman@your-file-system.com>
Date:   Tue Jul 3 15:38:33 2012 -0400

    Windows: RDR RXAFS_GetVolumeStatus vs 1.1 'l' only
    
    The Windows redirector relies upon the ability to obtain volume
    status information to decide whether a file system volume object
    can be created and whether or not an extending write can be
    permitted.  As of this writing, the file server always performs
    a PRFS_READ access check on the volume's root directory (1.1)
    vnode as a condition for releasing the volume state information
    which includes:
    
      Message of the day (if any)
      Offline message (if any)
      Online flag
      InService flag
      Blessed flag
      NeedsSalvage flag
      Type
      MinQuota
      MaxQuota
      BlocksInUse
      PartBlocksAvail
      PartMaxBlocks
    
    All of this information is publicly available to anonymous users
    via "vos examine" so it is odd that it is hidden from anonymous
    cache managers.  When RXAFS_GetVolumeStatus fails, the AFS redirector
    was failing to create a file system object for the AFS volume.  That
    in turn prevented the volume from being accessed even if the user
    had 'l' in the root directory and full access everywhere else.
    
    This patchset will make up fake data for the AFS volume if the
    RPC fails.  However, doing so does have consequences.  The client
    will be unable to make an accurate determination regarding free space
    on the file server.  As a result, an extending write may be permitted
    which writes data into the system page cache which in turn cannot
    be written to the file server.   Such data will be lost and unrecoverable.
    
    Reviewed-on: http://gerrit.openafs.org/7704
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
    Tested-by: Jeffrey Altman <jaltman@your-file-system.com>
    (cherry picked from commit 561b9ce80938d316223f2b313f8224fe06af9d5a)
    
    Change-Id: Ia8bce4e37d6b642b751c2c087ec95401a0e3681d
    Reviewed-on: http://gerrit.openafs.org/7715
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>

 src/WINNT/afsrdr/user/RDRFunction.c |  152 ++++++++++++++++++++---------------
 1 files changed, 86 insertions(+), 66 deletions(-)

-- 
OpenAFS Master Repository