OpenAFS Master Repository branch, master, updated. BP--openafs-stable-1_6_x-1533-g5c0e92a
Gerrit Code Review
gerrit@openafs.org
Sat, 13 Aug 2011 19:08:36 -0700 (PDT)
The following commit has been merged in the master branch:
commit 5c0e92a159f11366a88adab4700e3f909124a06a
Author: Garrett Wollman <wollman@csail.mit.edu>
Date: Sat Aug 13 19:01:26 2011 -0400
libafs: don't free a null pointer in an unlikely error condition
It is extremely unlikely that we will ever fail to allocate two
bytes in SRXAFSCB_GetCellServDB() to hold the empty-string return
value for the case where the specified cell can't be found. But
that would result in freeing a null pointer, so check for it.
Change-Id: I47a296148e231b0ef20ecd18b8458b912f22a58c
Found-by: clang static analyzer with the help of AFS_NONNULL
Reviewed-on: http://gerrit.openafs.org/5261
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@dementix.org>
src/afs/afs_callback.c | 3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
--
OpenAFS Master Repository