OpenAFS Master Repository branch, openafs-stable-1_6_x, updated. openafs-stable-1_6_0pre7-44-g3fd135d
Gerrit Code Review
gerrit@openafs.org
Fri, 12 Aug 2011 08:52:51 -0700 (PDT)
The following commit has been merged in the openafs-stable-1_6_x branch:
commit 3fd135d48b76b2e54329b75e53a67cac1abb29d4
Author: Garrett Wollman <wollman@csail.mit.edu>
Date: Sat Aug 6 23:15:14 2011 -0400
vos: don't free stack garbage on error
If wantExtendedInfo is true, then pntr is used uninitialized.
In the other case, UV_ListVolumes will have set it to NULL
before doing anything (even if it returns an error), so this
free() is dead anyway.
Found-by: clang static analyzer
Reviewed-on: http://gerrit.openafs.org/5164
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Simon Wilkinson <sxw@inf.ed.ac.uk>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
(cherry picked from commit 619b420903c99e48618db0d2f12085111573f279)
Change-Id: Icd76d3ac160ccb04b05445552a9e6d04f053efbb
Reviewed-on: http://gerrit.openafs.org/5227
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>
src/volser/vos.c | 2 --
1 files changed, 0 insertions(+), 2 deletions(-)
--
OpenAFS Master Repository