OpenAFS Master Repository branch, openafs-stable-1_6_x, updated. openafs-stable-1_6_8-67-g0fe14e5
Gerrit Code Review
gerrit@openafs.org
Fri, 6 Jun 2014 04:02:50 -0400
The following commit has been merged in the openafs-stable-1_6_x branch:
commit 0fe14e5487330a161dbbb1d293aba43ac395ddcf
Author: Simon Wilkinson <sxw@your-file-system.com>
Date: Sat Mar 2 11:09:08 2013 +0000
libadmin: Don't double check for required param
The server, process and stat_type parameters are declared as
required when the command syntax is set up - so they must be
present when the command handler is called. So, don't bother
checking for them.
Caught by coverity (#985963, #985964, #985965, #985966, #985967,
#985968, #985969, #985970, #985971, #985972,
#985973, #985974, #985975, #985976, #985977)
Reviewed-on: http://gerrit.openafs.org/9399
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
(cherry picked from commit f5b462c9d4a56be0ae4696db08fb92a1e7390e2c)
Change-Id: I4ea58eadbc97a934bb54a346086cdaf77ab28b9c
Reviewed-on: http://gerrit.openafs.org/11048
Reviewed-by: Andrew Deason <adeason@sinenomine.net>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
Reviewed-by: D Brashear <shadow@your-file-system.com>
Reviewed-by: Dan Hyde <drh@umich.edu>
Reviewed-by: Stephan Wiesand <stephan.wiesand@desy.de>
src/libadmin/test/client.c | 264 ++++++++++++++++++--------------------------
1 files changed, 108 insertions(+), 156 deletions(-)
--
OpenAFS Master Repository