OpenAFS Master Repository branch, openafs-stable-1_6_x, updated. openafs-stable-1_6_8-26-gb76b80b

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


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

    rxgen: Don't overflow PackageIndex
    
    PackageIndex++ returns the pre-index value of PackageIndex, so the
    error statement isn't run when PackageIndex == MAX_PACKAGES. This
    means we go on to overflow all of the arrays that are MAX_PACKAGES
    in size.
    
    Caught by coverity (#985583, #985584, #985585, #985586,
                        #985587, #985588, #985589)
    
    Reviewed-on: http://gerrit.openafs.org/9325
    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 a1d8109c8fa8c10e3ce5ed67cda03b3b557608ff)
    
    Change-Id: I5278bff5fe6be1bf127b240f7752c69385a9da24
    Reviewed-on: http://gerrit.openafs.org/11033
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Andrew Deason <adeason@sinenomine.net>
    Reviewed-by: Chas Williams - CONTRACTOR <chas@cmf.nrl.navy.mil>
    Reviewed-by: Stephan Wiesand <stephan.wiesand@desy.de>

 src/rxgen/rpc_parse.c |    5 ++++-
 1 files changed, 4 insertions(+), 1 deletions(-)

-- 
OpenAFS Master Repository