OpenAFS Master Repository branch, master, updated. openafs-devel-1_5_75-63-ga8b64db
Gerrit Code Review
gerrit@openafs.org
Wed, 14 Jul 2010 08:58:47 -0700 (PDT)
The following commit has been merged in the master branch:
commit a8b64dba013d5aa899d5ce29370bf1a52fb3fca9
Author: Jeffrey Hutzelman <jhutz@cmu.edu>
Date: Wed Jul 14 01:20:22 2010 -0400
Don't overflow a buffer on the server's stack at startup
The servers like to log the command line it was invoked with.
It does this by concatenating its arguments, separated by spaces,
into a 150-character buffer on the stack of main(). That's just
wrong. Use a dynamically-allocated buffer instead.
Change-Id: Ibe398e5f449eec176f11f301090a1b68ef13e94b
Reviewed-on: http://gerrit.openafs.org/2275
Tested-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Andrew Deason <adeason@sinenomine.net>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
src/ptserver/ptserver.c | 8 ++++++++
src/util/afsutil_prototypes.h | 3 +++
src/util/serverLog.c | 28 ++++++++++++++++++++++++++++
src/viced/viced.c | 6 +-----
src/vlserver/vldbint.xg | 2 ++
src/vlserver/vlserver.c | 11 ++---------
src/vol/salvaged.c | 31 ++++++++++++++++---------------
src/vol/salvager.c | 11 ++---------
src/volser/volmain.c | 11 ++---------
9 files changed, 64 insertions(+), 47 deletions(-)
--
OpenAFS Master Repository