The following commit has been merged in the master branch:
commit fda2bc874751ca479365dc6389c0eebb41a0bda1
Author: Jeffrey Hutzelman <[email protected]>
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.
[[email protected] convert to cleanup-handler memory management where
appropriate]
Change-Id: Ib1986187a1c467e867d50280aaf1d8a86d9108c8
Reviewed-on: http://gerrit.openafs.org/9985
Reviewed-by: Benjamin Kaduk <[email protected]>
Tested-by: BuildBot <[email protected]>
Reviewed-by: Michael Meffie <[email protected]>
Reviewed-by: Jeffrey Altman <[email protected]>
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
_______________________________________________
OpenAFS-cvs mailing list
[email protected]
https://lists.openafs.org/mailman/listinfo/openafs-cvs
