From: wang di <di.wang@intel.com>
Date: Thu, 10 Sep 2015 04:34:47 +0000 (-0700)
Subject: LU-7068 mdd: object leak in mdd_migrate_entries
X-Git-Tag: 2.7.64~25
X-Git-Url: https://git.whamcloud.com/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F84%2F16384%2F4;p=fs%2Flustre-release.git

LU-7068 mdd: object leak in mdd_migrate_entries

mdd_migrate_entries should release the object
if mdd_trans_start() fails.

Signed-off-by: wang di <di.wang@intel.com>
Change-Id: If10dce8b450e8c19bb93465beb08118e98c4ed96
Reviewed-on: http://review.whamcloud.com/16384
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Lai Siyao <lai.siyao@intel.com>
---

diff --git a/lustre/mdd/mdd_dir.c b/lustre/mdd/mdd_dir.c
index 430568d..db4ddad 100644
--- a/lustre/mdd/mdd_dir.c
+++ b/lustre/mdd/mdd_dir.c
@@ -3667,7 +3667,7 @@ static int mdd_migrate_entries(const struct lu_env *env,
 
 		handle = mdd_trans_create(env, mdd);
 		if (IS_ERR(handle))
-			GOTO(out, rc = PTR_ERR(handle));
+			GOTO(out_put, rc = PTR_ERR(handle));
 
 		/* Note: this transaction is part of migration, and it is not
 		 * the last step of migration, so we set th_local = 1 to avoid