I submitted the patch,
This is the test case to check,
From RO client if we try to append existing file, applications don't get any error even though write fails.

[root@centos71 tld]# cat a.txt

Mon Oct 22 11:33:46 EDT 2018

[root@centos71 tld]# echo "hi" >> a.txt

[root@centos71 tld]# cat a.txt

Mon Oct 22 11:33:46 EDT 2018


With the fix above it errors out with proper error.

[root@centos71 tld]# cat a.txt

Mon Oct 22 11:33:46 EDT 2018

[root@centos71 tld]# echo "hi" >> a.txt

bash: a.txt: Permission Denied


Regards,
Gaurav


On Tue, Oct 30, 2018 at 10:43 AM gaurav gangalwar <gaurav.gangalwar@gmail.com> wrote:
Sure, I will submit the patch.
Regards,
Gaurav


On Tue, Oct 30, 2018 at 1:01 AM Frank Filz <ffilzlnx@mindspring.com> wrote:

I think you’re right. Could you submit this patch to Gerrithub (see src/CONTRIBUTING_HOWTO.txt).

 

Frank

 

From: gaurav gangalwar [mailto:gaurav.gangalwar@gmail.com]
Sent: Sunday, October 28, 2018 10:50 PM
To: devel@lists.nfs-ganesha.org
Subject: [NFS-Ganesha-Devel] Access check issue for RO client.

 

Hi,

For client with ReadOnly access and user with ReadWrite access, Ganesha returns READ/MODIFY/APPEND allowed on ACCESS call.

Linux kernel NFS returns just READ allowed for same.

I think we need to fix Ganesha’s behaviour, we should return access rights based on client access check also.

Please check the patch attached to fix the behaviour.

Regards,

Gaurav