From: Oleg Drokin <green@whamcloud.com>
Date: Wed, 22 May 2019 05:22:49 +0000 (-0400)
Subject: LU-12324 mdd: Do not record xattr size get in changelogs
X-Git-Tag: 2.12.3-RC1~183
X-Git-Url: https://git.whamcloud.com/gitweb?a=commitdiff_plain;h=f94aecf16859e2e0dcd999eb8862227e50671ee8;p=fs%2Flustre-release.git

LU-12324 mdd: Do not record xattr size get in changelogs

It looks like if the xattr itself was not fetched there's no
need to create a changelog entry for it. The real get will come
and we'd do it there

Lustre-change: https://review.whamcloud.com/34936
Lustre-commit: 8f2599c599011a50549cdc79f0b3379239cf4f0c

Change-Id: I5b19f9309f65da0a4c58cb79a95787dab862eb94
Signed-off-by: Oleg Drokin <green@whamcloud.com>
Signed-off-by: Mikhail Pershin <mpershin@whamcloud.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Sebastien Buisson <sbuisson@ddn.com>
Signed-off-by: Minh Diep <mdiep@whamcloud.com>
Reviewed-on: https://review.whamcloud.com/35010
Tested-by: Jenkins
Tested-by: Maloo <maloo@whamcloud.com>
---

diff --git a/lustre/mdd/mdd_object.c b/lustre/mdd/mdd_object.c
index 669be61..312c5f8 100644
--- a/lustre/mdd/mdd_object.c
+++ b/lustre/mdd/mdd_object.c
@@ -428,7 +428,7 @@ static int mdd_xattr_get(const struct lu_env *env,
 	mdd = mdo2mdd(obj);
 
 	/* record only getting user xattrs and acls */
-	if (rc >= 0 &&
+	if (rc >= 0 && buf->lb_buf &&
 	    mdd_changelog_enabled(env, mdd, CL_GETXATTR) &&
 	    (has_prefix(name, XATTR_USER_PREFIX) ||
 	     has_prefix(name, XATTR_NAME_POSIX_ACL_ACCESS) ||