OpenAFS Master Repository branch, master, updated. BP-openafs-stable-1_8_x-412-gbf24b30
Gerrit Code Review
gerrit@openafs.org
Fri, 2 Aug 2019 13:29:51 -0400
The following commit has been merged in the master branch:
commit bf24b301a10dcb5710a98e58252213bd72c6f352
Author: Cheyenne Wills <cwills@sinenomine.net>
Date: Fri Aug 2 10:31:13 2019 -0600
restorevol: replace snprintf with asprintf
GCC is generating format-truncations warnings. With newer levels of gcc
(e.g. gcc8) and --checking-enabled these warnings result in errors and
failed builds. In addition clang8 static analysis tools are reporting
memory leaks.
Replace snprintf with asprintf and eliminate some of the large work
buffers that are being placed on the stack. In order to correct some of
the format-truncation errors the size of the buffers grew significantly
(e.g. gcc is reporting the need to resize some of the buffers from 256
bytes to 4K in order to eliminate the warnings).
Ensure allocated work buffers are freed before function return.
Obtained a clean build with gcc9/clang8 with --enable-checking and a
clean scan-build report with clang8.
Change-Id: Ie8e22fdff2e0ba6494b1b449f413ecbe38f367bd
Reviewed-on: https://gerrit.openafs.org/13494
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
src/volser/restorevol.c | 316 ++++++++++++++++++++++++++++++++++++-----------
1 files changed, 241 insertions(+), 75 deletions(-)
--
OpenAFS Master Repository