OpenAFS Master Repository branch, master, updated. openafs-devel-1_5_76-3724-ged76e57

Gerrit Code Review
Fri, 26 Apr 2013 06:23:35 -0700 (PDT)

The following commit has been merged in the master branch:
commit a1aa06e82ca770456884b1b96b4f0c109cd06a85
Author: Jeffrey Altman <>
Date:   Fri Apr 12 10:58:47 2013 -0400

    Windows: AFSInitDirEntry allocated ObjInfoCBs valid
    In AFSInitDirEntry the pattern was to find or allocate an
    ObjectInfoCB then destroy it if the DirectoryCB creation fails
    for some reason.  The problem with this approach is that once the
    VolumeCB ObjectInfoTree.TreeLock is dropped the ObjectInfoCB is findable.
    That means that the contents of the ObjectInfoCB must be valid.
    This patchset makes three changes.  First, in the case where the
    ObjectInfoCB is allocated, the fields of the ObjectInfoCB are populated
    from the DirEnumEntry before the TreeLock is dropped.  Second, if the
    DirectoryCB allocation fails the ObjectInfoCB is not deleted.  It is
    perfectly valid and can be used by a subsequent AFSInitDirEntry call.
    Perhaps one that is racing with this thread.   It will eventually be
    cleaned up by the AFSPrimaryVolumeWorkerThread.  Finally, when the
    ObjectInfoCB reference count is decremented the TreeLock is held shared in
    order to prevent races with other threads that might be incrementing it
    Change-Id: If3091d4fa640bbb614a1a405c3afc910d649aad6
    Reviewed-by: Rod Widdowson <>
    Tested-by: BuildBot <>
    Reviewed-by: Jeffrey Altman <>

 src/WINNT/afsrdr/kernel/lib/AFSGeneric.cpp |  193 +++++++++++++---------------
 1 files changed, 92 insertions(+), 101 deletions(-)

OpenAFS Master Repository