OpenAFS Master Repository branch, master, updated. openafs-devel-1_5_76-3958-g1cf6678

Gerrit Code Review gerrit@openafs.org
Mon, 30 Sep 2013 07:51:03 -0700 (PDT)


The following commit has been merged in the master branch:
commit 1cf6678fdaae82871a9aeb4addfed3a2db1954e8
Author: Andrew Deason <adeason@sinenomine.net>
Date:   Thu Sep 12 15:58:34 2013 -0500

    ihandle: Make sure we don't ih_attachfd invalid FD
    
    Right now, if you give ih_attachfd_r an invalid fd, and fdLruHead is
    NULL, we'll return an FdHandle_t* for an invalid fd. Nowhere in the
    code is this possible right now, but the implementation of
    ih_attachfd_r and ih_attachfd doesn't make this very clear.
    
    Ideally the "close some fds and retry" behavior in ih_attachfd_r will
    be split out, so this code could be easier to follow, and we could
    implement open() EMFILE retrying for icreate operations. But for now,
    just make the current behavior clearer, so future modifications do not
    introduce such mistakes.
    
    Change-Id: Ibc80b32bc6f50480d12e3241fe198bc0587a962c
    Reviewed-on: http://gerrit.openafs.org/10249
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Derrick Brashear <shadow@your-file-system.com>

 src/vol/ihandle.c |   22 +++++++++++++++-------
 1 files changed, 15 insertions(+), 7 deletions(-)

-- 
OpenAFS Master Repository