From c8cbf38929f9f60cef75ff11a0dd1cc6f74cd533 Mon Sep 17 00:00:00 2001 From: nasf Date: Wed, 4 May 2011 21:43:11 +0800 Subject: [PATCH] LU-271 Skip permission check for NFS open after create For open_create operation, NFS client will trigger separate open RPC after create RPC. We only need to check create permission for such case. Change-Id: I7f0e2edb1b14f1044790180345f2c8328d92d4e8 Signed-off-by: nasf Reviewed-on: http://review.whamcloud.com/494 Tested-by: Hudson Reviewed-by: Oleg Drokin Reviewed-by: Lai Siyao --- lustre/llite/file.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lustre/llite/file.c b/lustre/llite/file.c index 5af347b..d316e18 100644 --- a/lustre/llite/file.c +++ b/lustre/llite/file.c @@ -539,7 +539,7 @@ int ll_file_open(struct inode *inode, struct file *file) * dentry_open after call to open_namei that checks permissions. * Only nfsd_open call dentry_open directly without checking * permissions and because of that this code below is safe. */ - if (oit.it_flags & FMODE_WRITE) + if (oit.it_flags & (FMODE_WRITE | FMODE_READ)) oit.it_flags |= MDS_OPEN_OWNEROVERRIDE; /* We do not want O_EXCL here, presumably we opened the file -- 1.8.3.1