OpenAFS Master Repository branch, openafs-stable-1_6_x, updated. openafs-stable-1_6_8-9-gadc3464

Gerrit Code Review gerrit@openafs.org
Tue, 3 Jun 2014 12:15:35 -0400


The following commit has been merged in the openafs-stable-1_6_x branch:
commit adc34642923d8969c3aa6bc73d90e0817ba57a8e
Author: Simon Wilkinson <sxw@your-file-system.com>
Date:   Tue Feb 26 21:28:52 2013 +0000

    volser: Fix bad readlink usage
    
    readlink fills the buffer passed to it with a non-terminated string.
    It can legitimately fill the whole of this buffer. So, if we require
    a string to be NUL terminated, we must give readlink one less than
    the string length so that the termination character can be safely
    appended.
    
    Caught by coverity (#985611)
    
    Reviewed-on: http://gerrit.openafs.org/9290
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
    Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
    (cherry picked from commit ddc37043351056c402158610477312f0d7d01c13)
    
    Change-Id: Ic3aa9f767b3dce988e32ce670763791f17e72aa2
    Reviewed-on: http://gerrit.openafs.org/11017
    Reviewed-by: Chas Williams - CONTRACTOR <chas@cmf.nrl.navy.mil>
    Reviewed-by: Andrew Deason <adeason@sinenomine.net>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
    Reviewed-by: Stephan Wiesand <stephan.wiesand@desy.de>

 src/volser/restorevol.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

-- 
OpenAFS Master Repository