From: Steven Whitehouse <swhiteho@redhat.com>
Date: Fri, 16 Aug 2013 20:10:28 +0000 (+0100)
Subject: GFS2: Check for glock already held in gfs2_getxattr
X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=7bd9ee58a4fe026514266757e812cb89c3c945eb;p=linux.git

GFS2: Check for glock already held in gfs2_getxattr

Since the introduction of atomic_open, gfs2_getxattr can be
called with the glock already held, so we need to allow for
this.

Signed-off-by: Steven Whitehouse <swhiteho@redhat.com>
Reported-by: David Teigland <teigland@redhat.com>
Tested-by: David Teigland <teigland@redhat.com>
---

diff --git a/fs/gfs2/inode.c b/fs/gfs2/inode.c
index a01b8fd3a1c16..64915eeae5a71 100644
--- a/fs/gfs2/inode.c
+++ b/fs/gfs2/inode.c
@@ -1750,6 +1750,10 @@ static ssize_t gfs2_getxattr(struct dentry *dentry, const char *name,
 	struct gfs2_holder gh;
 	int ret;
 
+	/* For selinux during lookup */
+	if (gfs2_glock_is_locked_by_me(ip->i_gl))
+		return generic_getxattr(dentry, name, data, size);
+
 	gfs2_holder_init(ip->i_gl, LM_ST_SHARED, LM_FLAG_ANY, &gh);
 	ret = gfs2_glock_nq(&gh);
 	if (ret == 0) {