From: Parinay Kondekar <parinay.kondekar@seagate.com>
Date: Mon, 18 Jan 2016 03:29:35 +0000 (+0530)
Subject: LU-6512 llite:ll_dir_ioctl cleanup of redundant comparisons
X-Git-Tag: 2.8.53~23
X-Git-Url: https://git.whamcloud.com/?a=commitdiff_plain;h=c341ac9083c1764675c92363e1dd8bcd3223788a;p=fs%2Flustre-release.git

LU-6512 llite:ll_dir_ioctl cleanup of redundant comparisons

In ll_dir_ioctl() two identical comparisions are present for
return code (rc) of ll_dir_getstripe(). This patch removes
the other inside if( ) condition which is not necessary.

Change-Id: Ia32dd5ac5937d30dd3676094ef99a623e5cca608
Signed-off-by: Parinay Kondekar <parinay.kondekar@seagate.com>
Reviewed-on: http://review.whamcloud.com/18027
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Bobi Jam <bobijam@hotmail.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Tested-by: Jenkins
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
---

diff --git a/lustre/llite/dir.c b/lustre/llite/dir.c
index 4f28145..ec71a19 100644
--- a/lustre/llite/dir.c
+++ b/lustre/llite/dir.c
@@ -1293,9 +1293,6 @@ lmv_out_free:
 
 		/* Get default LMV EA */
 		if (lum.lum_magic == LMV_USER_MAGIC) {
-			if (rc != 0)
-				GOTO(finish_req, rc);
-
 			if (lmmsize > sizeof(*ulmv))
 				GOTO(finish_req, rc = -EINVAL);