summaryrefslogtreecommitdiffstats
path: root/drivers/staging/lustre
diff options
context:
space:
mode:
authorSandhya Bankar <bankarsandhya512@gmail.com>2016-09-18 05:23:49 +0530
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2016-09-20 13:23:40 +0200
commit83a75e3f14667b334ecd53b6334407ace8679eed (patch)
treeeeacaae6968974c55625e60454545dcd9df090ef /drivers/staging/lustre
parent7d7005180a7eab473bca4c1fe807f0589561791e (diff)
downloadop-kernel-dev-83a75e3f14667b334ecd53b6334407ace8679eed.zip
op-kernel-dev-83a75e3f14667b334ecd53b6334407ace8679eed.tar.gz
Staging: lustre: Convert array index from the loop bound to the loop index.
Convert array index from the loop bound to the loop index. The structure cl_env_percpu[NR_CPUS] has been initializing for each possible cpu (i.e 0 to i). During initialization if any error will occurred, the error handling code should uninitialize cl_env_percpu upto i. But currently unitialization is repeatedly done for the same cl_env_percpu[i] element. This does not seems to be correct. Used below semantic patch to identify this issue: @@ expression e1,e2,ar; @@ for(e1 = 0; e1 < e2; e1++) { <... ar[ - e2 + e1 ] ...> } Signed-off-by: Sandhya Bankar <bankarsandhya512@gmail.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/lustre')
-rw-r--r--drivers/staging/lustre/lustre/obdclass/cl_object.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/lustre/lustre/obdclass/cl_object.c b/drivers/staging/lustre/lustre/obdclass/cl_object.c
index e493c80..3199dd4 100644
--- a/drivers/staging/lustre/lustre/obdclass/cl_object.c
+++ b/drivers/staging/lustre/lustre/obdclass/cl_object.c
@@ -1022,7 +1022,7 @@ static int cl_env_percpu_init(void)
* thus we must uninitialize up to i, the rest are undefined.
*/
for (j = 0; j < i; j++) {
- cle = &cl_env_percpu[i];
+ cle = &cl_env_percpu[j];
lu_context_exit(&cle->ce_ses);
lu_context_fini(&cle->ce_ses);
lu_env_fini(&cle->ce_lu);
OpenPOWER on IntegriCloud