svn commit: r323689 - head/sys/fs/nfsclient

Rick Macklem rmacklem at FreeBSD.org
Sun Sep 17 22:18:02 UTC 2017


Author: rmacklem
Date: Sun Sep 17 22:18:01 2017
New Revision: 323689
URL: https://svnweb.freebsd.org/changeset/base/323689

Log:
  Fix bogus FREAD with NFSV4OPEN_ACCESSREAD. No functional change.
  
  The code in nfscl_doflayoutio() bogusly used FREAD instead of
  NFSV4OPEN_ACCESSREAD. Since both happen to be defined as "1", this
  worked and the patch doesn't result in a functional change.
  Found by inspection during development of Flex File Layout support.
  
  MFC after:	2 weeks

Modified:
  head/sys/fs/nfsclient/nfs_clrpcops.c

Modified: head/sys/fs/nfsclient/nfs_clrpcops.c
==============================================================================
--- head/sys/fs/nfsclient/nfs_clrpcops.c	Sun Sep 17 21:41:51 2017	(r323688)
+++ head/sys/fs/nfsclient/nfs_clrpcops.c	Sun Sep 17 22:18:01 2017	(r323689)
@@ -5674,7 +5674,7 @@ nfscl_doflayoutio(vnode_t vp, struct uio *uiop, int *i
 				np->n_flag &= ~NDSCOMMIT;
 				mtx_unlock(&np->n_mtx);
 			}
-		} else if (rwflag == FREAD)
+		} else if (rwflag == NFSV4OPEN_ACCESSREAD)
 			error = nfsrpc_readds(vp, uiop, stateidp, eofp, *dspp,
 			    io_off, xfer, fhp, cred, p);
 		else {


More information about the svn-src-all mailing list