Re: #255931 NFS fh_verify causes lost output

[Olaf Kirch <okir@suse.de>]

[-- Attachment #1: Type: text/plain, Size: 577 bytes --]

On Wed, Aug 18, 2004 at 09:00:41PM +0100, Christoph Hellwig wrote:
> We have the following report from Paul Szabo in the Debian bugtracking system,
> any chance someone could look into it?  The RedHat bugzilla reference is quite
> interesting aswell.

I reported exactly this problem here a few months ago; the fix I proposed
was to return EACCES rather than ESTALE in find_exported_dentry()
(fs/exportfs/expfs.c). Not sure what happened to this patch; seems it
fell through the cracks.

Olaf
-- 
Olaf Kirch     |  The Hardware Gods hate me.
okir@suse.de   |
---------------+ 

[-- Attachment #2: subtreecheck-nostale.patch --]
[-- Type: text/plain, Size: 657 bytes --]

--- linux-2.6.5/fs/exportfs/expfs.c.nostale	2004-04-04 05:37:44.000000000 +0200
+++ linux-2.6.5/fs/exportfs/expfs.c	2004-04-30 10:32:19.000000000 +0200
@@ -278,7 +278,15 @@
 
 	/* drat - I just cannot find anything acceptable */
 	dput(result);
-	return ERR_PTR(-ESTALE);
+
+	/* Originally, we would return ESTALE here. This is not
+	 * correct, however, as the file handle is valid; it just
+	 * failed our acceptance test.
+	 * This will lead to unexpected results in the client if
+	 * there are two processes accessing the same file; one
+	 * with proper permissions and one without.
+	 */
+	return ERR_PTR(-EACCES);
 
  err_target:
 	dput(target_dir);