OpenAFS Master Repository branch, master, updated. openafs-devel-1_5_76-2783-gff217de
Gerrit Code Review
gerrit@openafs.org
Thu, 26 Jul 2012 11:44:21 -0700 (PDT)
The following commit has been merged in the master branch:
commit ff217dec93c9b46a2f161bea5439d5a41337bf51
Author: Garrett Wollman <wollman@csail.mit.edu>
Date: Wed Jul 25 00:22:10 2012 -0400
afsdump_extract: clarify logic to avoid freeing local buffer
Sometimes vnodepath is set to a local buffer. Sometimes it is set
to malloc'ed storage. Simplify the logic for freeing vnodepath
by checking explicitly for this condition rather than the state
of other variables. As a bonus, avoids a false (?) positive from
the static analyzer.
Change-Id: I3772cb97698acc5a6ac1f438977c673e6fea7722
Reviewed-on: http://gerrit.openafs.org/7869
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Simon Wilkinson <simonxwilkinson@gmail.com>
Reviewed-by: Alistair Ferguson <alistair.ferguson@mac.com>
Reviewed-by: Derrick Brashear <shadow@dementix.org>
src/tools/dumpscan/afsdump_extract.c | 10 +++++-----
1 files changed, 5 insertions(+), 5 deletions(-)
--
OpenAFS Master Repository