OpenAFS Master Repository branch, master, updated. openafs-devel-1_5_76-4614-gfda2bc8
Gerrit Code Review
gerrit@openafs.org
Thu, 23 Apr 2015 00:52:55 -0400
The following commit has been merged in the master branch:
commit fda2bc874751ca479365dc6389c0eebb41a0bda1
Author: Jeffrey Hutzelman <jhutz@cmu.edu>
Date: Mon Jun 17 21:08:14 2013 -0400
Allocate pathname buffers dynamically
This change reworks numerous places which formerly used potentially
large on-stack buffers (of size AFSDIR_PATH_MAX) for constructing or
storing pathnames. Instead, these buffers are now allocated from the
heap, either by using asprintf() to build a pathname in a correctly
sized buffer or, where necessary, using malloc() to allocate a buffer
of size AFSDIR_PATH_MAX.
A few occurrances of AFSDIR_PATH_MAX-sized buffers are not changed;
these are generally either globals or are contained within another
data structure that is already allocated on the heap.
[kaduk@mit.edu convert to cleanup-handler memory management where
appropriate]
Change-Id: Ib1986187a1c467e867d50280aaf1d8a86d9108c8
Reviewed-on: http://gerrit.openafs.org/9985
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Michael Meffie <mmeffie@sinenomine.net>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
src/WINNT/afsreg/test/regman.c | 39 ++++++++----
src/bozo/bosoprocs.c | 135 +++++++++++++++++++++++++---------------
src/bozo/bosserver.c | 39 ++++++++----
src/bozo/fsbnodeops.c | 120 +++++++++++++++++++++++++----------
src/bucoord/dsstub.c | 35 +++++++----
src/butc/tcmain.c | 6 +-
src/kauth/kaserver.c | 10 ++-
src/kauth/kdb.c | 7 +-
src/libadmin/cfg/cfgservers.c | 117 ++++++++++-------------------------
src/ptserver/pts.c | 8 ++-
src/update/client.c | 50 ++++++++++++---
src/update/server.c | 48 ++++++++++++---
src/vol/daemon_com.c | 10 +--
src/vol/salvaged.c | 36 ++++++-----
src/vol/vol-salvage.c | 38 ++++++-----
15 files changed, 421 insertions(+), 277 deletions(-)
--
OpenAFS Master Repository