OpenAFS Master Repository branch, master, updated. openafs-devel-1_5_76-3624-g8b76110

Gerrit Code Review gerrit@openafs.org
Tue, 12 Mar 2013 06:52:23 -0700 (PDT)


The following commit has been merged in the master branch:
commit 8b76110e23160e33716234f9560d22ca5275cb0a
Author: Simon Wilkinson <sxw@your-file-system.com>
Date:   Fri Mar 8 16:15:51 2013 +0000

    bos: Remove theoretical overflow in DateOf
    
    DateOf copies the results of ctime into a static buffer. Typically
    ctime will return a 26 byte string, but if you pass it a year larger
    than 9999 (which we shouldn't), you can get a 32 (or more) byte string.
    
    Get rid of this unlikely event by using strlcpy for the copy. We already
    truncate at 24 bytes when we remove the \n, so this shouldn't cause any
    further problems.
    
    Really, this whole thing should be rewritten to use strftime.
    
    Caught by coverity (#985776)
    
    Change-Id: I18f6828d6ec3d79ecaf1dad8e27d3e8691ce87d5
    Reviewed-on: http://gerrit.openafs.org/9551
    Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
    Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>

 src/bozo/bos.c |    7 +++----
 1 files changed, 3 insertions(+), 4 deletions(-)

-- 
OpenAFS Master Repository