OpenAFS Master Repository branch, openafs-stable-1_8_x, updated. openafs-stable-1_8_0pre2-14-ge2c47ca

Gerrit Code Review gerrit@openafs.org
Tue, 28 Nov 2017 11:46:32 -0500


The following commit has been merged in the openafs-stable-1_8_x branch:
commit e2c47cae56ba0d804af119fb158a9fe77fa3a15e
Author: Benjamin Kaduk <kaduk@mit.edu>
Date:   Mon Nov 27 22:17:28 2017 -0600

    afs: Fix bounds check in PNewCell
    
    Reported by the opensuse buildbot:
    
      CC [M]  /home/buildbot/opensuse-tumbleweed-i386-builder/build/src/libafs/MODLOAD-4.13.12-1-default-MP/rx_packet.o
    /home/buildbot/opensuse-tumbleweed-i386-builder/build/src/afs/afs_pioctl.c: In function ‘PNewCell’:
    /home/buildbot/opensuse-tumbleweed-i386-builder/build/src/afs/afs_pioctl.c:3075:55: error: ‘*’ in boolean context, suggest ‘&&’ instead [-Werror=int-in-bool-context]
         if ((afs_pd_remaining(ain) < AFS_MAXCELLHOSTS +3) * sizeof(afs_int32))
             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~
    
    The bug was introduced in commit 718f85a8b6.
    
    Reviewed-on: https://gerrit.openafs.org/12782
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
    (cherry picked from commit 4fa0ee620cfb9991ca9748b5ee116cc8e1e6c505)
    
    Change-Id: I0963403846a62dddf2d13ce3c03d772a6d869119
    Reviewed-on: https://gerrit.openafs.org/12784
    Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>
    Tested-by: BuildBot <buildbot@rampaginggeek.com>
    Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>

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

-- 
OpenAFS Master Repository