OpenAFS Master Repository branch, master, updated. BP--openafs-stable-1_6_x-1343-g39083fe

Gerrit Code Review gerrit@openafs.org
Wed, 6 Jul 2011 13:42:15 -0700 (PDT)


The following commit has been merged in the master branch:
commit 39083fe1edab784fcd75eacbdaaf7f6affa14c9f
Author: Andrew Deason <adeason@sinenomine.net>
Date:   Wed Jun 22 13:44:38 2011 -0500

    afs: Ensure afs_calc_inum yields nonzero ino
    
    afs_calc_inum can currently yield an inode of 0 if MD5-based inode
    numbers are turned on. Some userspace applications (and for some
    platforms, maybe even the kernel) make certain assumptions about the
    inode number for a file; in particular for example, 'ls' will not
    display a file with inode 0 in a normal directory listing.
    
    So, read the md5 digest until we get a non-zero result. Fall back to
    the non-md5 calculation if we still somehow end up with a 0.
    
    While this case may at first glance seem to be extremely rare, in
    practice it can occur, as the current calculation for volume
    538313506, vnode 26178 does actually yield a 0.
    
    Change-Id: Iee1ef4cc2ad66269f2c677e29d586ef0964d7c70
    Reviewed-on: http://gerrit.openafs.org/4901
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>

 src/afs/afs_util.c |   22 ++++++++++++++++++----
 1 files changed, 18 insertions(+), 4 deletions(-)

-- 
OpenAFS Master Repository