OpenAFS Master Repository branch, openafs-stable-1_6_x, updated. openafs-stable-1_6_22_2-36-gb4ad4b0

Gerrit Code Review gerrit@openafs.org
Fri, 4 May 2018 04:35:02 -0400


The following commit has been merged in the openafs-stable-1_6_x branch:
commit b4ad4b0ad4eb88f680578204f1b5f38189cfdb38
Author: Michael Meffie <mmeffie@sinenomine.net>
Date:   Tue Feb 20 11:51:01 2018 -0500

    afs: improve -volume-ttl error messages
    
    Change the afs call which sets the volume ttl value to return EFAULT
    instead of EINVAL when given an out of range value for the volume ttl
    parameter.  This is more consistent with the other op codes, which
    return EFAULT when given an out of range parameter and allows the caller
    to distinguish between an invalid opcode and a bad parameter.
    
    Move the volume ttl range constants to afs_args.h, which is where
    constants related to the op codes are supposed to be defined. This makes
    the constants available to the caller in afsd.c as well as the
    implementation in afs_call.c.
    
    Update afsd to print a more sensible error message when the volume ttl
    set calls fails due to an out of range parameter.
    
    Reviewed-on: https://gerrit.openafs.org/12918
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
    (cherry picked from commit 6d74e3d6a1becf86cec30efc2d01a5692167afe1)
    
    Reviewed-on: https://gerrit.openafs.org/12936
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Mark Vitale <mvitale@sinenomine.net>
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
    (cherry picked from commit 0b1d10fd2535b0059d1e88c23fbd3f60041edc9f)
    
    Change-Id: I3ed43d819e4d1b5aff7e9108fc485b2358dd6d25
    Reviewed-on: https://gerrit.openafs.org/13023
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>
    Reviewed-by: Andrew Deason <adeason@sinenomine.net>
    Reviewed-by: Marcio Brito Barbosa <mbarbosa@sinenomine.net>
    Reviewed-by: Mark Vitale <mvitale@sinenomine.net>
    Reviewed-by: Stephan Wiesand <stephan.wiesand@desy.de>

 src/afs/afs.h         |    4 ----
 src/afs/afs_call.c    |    2 +-
 src/afsd/afsd.c       |   16 ++++++++++++++--
 src/config/afs_args.h |    3 +++
 4 files changed, 18 insertions(+), 7 deletions(-)

-- 
OpenAFS Master Repository