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 {
_______________________________________________
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to