From: John L. Hammond <john.hammond@intel.com>
Date: Wed, 23 Jul 2014 23:38:53 +0000 (-0500)
Subject: LU-5401 mdt: handle getattr errors in mdt_reint_open()
X-Git-Tag: 2.6.52~29
X-Git-Url: https://git.whamcloud.com/?a=commitdiff_plain;h=1f2f41a1cb893ce45cf7d5d7440c536455f66686;p=fs%2Flustre-release.git

LU-5401 mdt: handle getattr errors in mdt_reint_open()

In mdt_reint_open() if mdt_attr_get_complex_fails() then bail out and
return an error.

Signed-off-by: John L. Hammond <john.hammond@intel.com>
Change-Id: I67c773b93100064e4cfdc82b4356424fd102c925
Reviewed-on: http://review.whamcloud.com/11210
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Fan Yong <fan.yong@intel.com>
Reviewed-by: Mike Pershin <mike.pershin@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
---

diff --git a/lustre/mdt/mdt_open.c b/lustre/mdt/mdt_open.c
index 87354a2..50a8e61 100644
--- a/lustre/mdt/mdt_open.c
+++ b/lustre/mdt/mdt_open.c
@@ -1825,6 +1825,8 @@ int mdt_reint_open(struct mdt_thread_info *info, struct mdt_lock_handle *lhc)
 			mdt_prep_ma_buf_from_rep(info, child, ma);
 			ma->ma_need |= MA_HSM;
 			result = mdt_attr_get_complex(info, child, ma);
+			if (result != 0)
+				GOTO(out_child, result);
 		} else {
 			/* Object does not exist. Likely FS corruption. */
 			CERROR("%s: name '"DNAME"' present, but FID "