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