]> git.michaelhowe.org Git - packages/o/openafs.git/commit
libadmin: Don't double check for required param
authorSimon Wilkinson <sxw@your-file-system.com>
Sat, 2 Mar 2013 11:09:08 +0000 (11:09 +0000)
committerStephan Wiesand <stephan.wiesand@desy.de>
Fri, 6 Jun 2014 08:02:32 +0000 (04:02 -0400)
commit0fe14e5487330a161dbbb1d293aba43ac395ddcf
treeb6a98101ca0ca8a2061057e38a99dc8efb3b0584
parente47d97c8ae316c90ecb636a497c52cdb9a27a935
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