diff options
| author |   Sebastien Buisson <sbuisson@ddn.com> | 2017-07-26 11:22:20 -0400 |
|---|---|---|
| committer |   Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-07-30 08:06:10 -0700 |
| commit | fb04f121db31f450afaf344ff4b72cd1735b8646 (patch) | |
| tree | ce6ec53f73638795f25e6e8a053397e9fd62daa2 /drivers/staging/lustre/lustre/lov/lov_io.c | |
| parent | staging: lustre: ldlm: crash on umount in cleanup_resource (diff) | |
| download | linux-dev-fb04f121db31f450afaf344ff4b72cd1735b8646.tar.xz linux-dev-fb04f121db31f450afaf344ff4b72cd1735b8646.zip | |
staging: lustre: lov: fix 'control flow' error in lov_io_init_released
Fix "control flow" issues found by Coverity version 6.6.1:
Logically dead code (DEADCODE)
Execution cannot reach this statement.
Signed-off-by: Sebastien Buisson <sbuisson@ddn.com>
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-4048
Reviewed-on: http://review.whamcloud.com/7824
Reviewed-by: Jinshan Xiong <jinshan.xiong@intel.com>
Reviewed-by: Emoly Liu <emoly.liu@intel.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
Signed-off-by: James Simmons <jsimmons@infradead.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/lustre/lustre/lov/lov_io.c')
| -rw-r--r-- | drivers/staging/lustre/lustre/lov/lov_io.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/staging/lustre/lustre/lov/lov_io.c b/drivers/staging/lustre/lustre/lov/lov_io.c index babf39adef85..3e373535e6c0 100644 --- a/drivers/staging/lustre/lustre/lov/lov_io.c +++ b/drivers/staging/lustre/lustre/lov/lov_io.c @@ -1047,6 +1047,8 @@ int lov_io_init_released(const struct lu_env *env, struct cl_object *obj, switch (io->ci_type) { default: LASSERTF(0, "invalid type %d\n", io->ci_type); + result = -EOPNOTSUPP; + break; case CIT_MISC: case CIT_FSYNC: case CIT_DATA_VERSION: |