OpenAFS Master Repository branch, openafs-stable-1_4_x, updated. 2d6c0390f8ca702c05cb0c13e0994751a60285f9

Gerrit Code Review gerrit@openafs.org
Fri, 20 Nov 2009 12:49:25 -0800 (PST)


The following commit has been merged in the openafs-stable-1_4_x branch:
commit 2d6c0390f8ca702c05cb0c13e0994751a60285f9
Author: Dan Hyde <drh@umich.edu>
Date:   Thu Oct 29 12:07:47 2009 -0400

    Add array bounds checking in h_Enumerate
    
    When hostList is not properly NULL-terminated, the current code does
    not protect from buffer overflow.  The following patch prevents buffer
    overflow, prints a message, and asserts.
    
    On our Linux hosts, we never reached the original assert, as there is
    a problem handling the segfault the buffer overflow causes.
    
    FIXES 125506
    
    Reviewed-on: http://gerrit.openafs.org/757
    Reviewed-by: Dan Hyde <drh@umich.edu>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>
    Tested-by: Derrick Brashear <shadow@dementia.org>
    (cherry picked from commit 93d48b1735b88b3051357307bdbebf5da9872d69)
    
    Change-Id: Idb920d2bd309b40ae9bebc6972e614ddf9ba8dcc
    Reviewed-on: http://gerrit.openafs.org/862
    Tested-by: Andrew Deason <adeason@sinenomine.net>
    Reviewed-by: Derrick Brashear <shadow@dementia.org>

 src/viced/host.c |    6 ++++--
 1 files changed, 4 insertions(+), 2 deletions(-)

-- 
OpenAFS Master Repository