OpenAFS Master Repository branch, openafs-stable-1_6_x, updated. openafs-stable-1_6_2-25-g9fc38b1

Gerrit Code Review gerrit@openafs.org
Fri, 22 Mar 2013 06:00:06 -0700 (PDT)


The following commit has been merged in the openafs-stable-1_6_x branch:
commit 9fc38b1bcdfd7c16fc84f88314d7b9ec0f084b18
Author: Simon Wilkinson <sxw@your-file-system.com>
Date:   Fri Mar 1 11:47:03 2013 +0000

    ubik: Don't overflow server's addr array
    
    We're checking to see if we've overflowed the array _after_ we've
    looked up an element within it - so on the final iteration, we
    always read past the end of the array.
    
    Fix this by swapping the order of the tests in the for statemen
    
    Caught by coverity (#985590)
    
    Reviewed-on: http://gerrit.openafs.org/9326
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
    Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
    (cherry picked from commit a4ae9a4c3cc1fa5ca2b813174040dce36284a4b9)
    
    Conflicts:
    	src/ubik/recovery.c
    
    Change-Id: Ic09177a1e85492e181e51d554ddb25df2ca2dfbd
    Reviewed-on: http://gerrit.openafs.org/9373
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Andrew Deason <adeason@sinenomine.net>
    Reviewed-by: Paul Smeddle <paul.smeddle@gmail.com>
    Reviewed-by: Stephan Wiesand <stephan.wiesand@desy.de>

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

-- 
OpenAFS Master Repository