diff options
| author | rmacklem <rmacklem@FreeBSD.org> | 2014-12-31 00:44:11 +0000 |
|---|---|---|
| committer | rmacklem <rmacklem@FreeBSD.org> | 2014-12-31 00:44:11 +0000 |
| commit | 27e956ad1b0a1f1b86e8038a8be1b31a7d7f5877 (patch) | |
| tree | 8c7b76eea4201c940e4b7ba62484ffd65214cb35 /sys/fs | |
| parent | f40a1cfb98bbad16f8e8d9b1adb4fdc8109a2139 (diff) | |
| download | FreeBSD-src-27e956ad1b0a1f1b86e8038a8be1b31a7d7f5877.zip FreeBSD-src-27e956ad1b0a1f1b86e8038a8be1b31a7d7f5877.tar.gz | |
MFC: r276221
Delete some duplicate code that was harmless because
exactly the same code is at the end of the nfscl_checksattr()
function that is called just before it. As such, this code
had already been executed and didn't do anything.
Diffstat (limited to 'sys/fs')
| -rw-r--r-- | sys/fs/nfsclient/nfs_clvnops.c | 14 |
1 files changed, 0 insertions, 14 deletions
diff --git a/sys/fs/nfsclient/nfs_clvnops.c b/sys/fs/nfsclient/nfs_clvnops.c index c24d58b..4198494 100644 --- a/sys/fs/nfsclient/nfs_clvnops.c +++ b/sys/fs/nfsclient/nfs_clvnops.c @@ -1608,20 +1608,6 @@ again: } } else if (NFS_ISV34(dvp) && (fmode & O_EXCL)) { if (nfscl_checksattr(vap, &nfsva)) { - /* - * We are normally called with only a partially - * initialized VAP. Since the NFSv3 spec says that - * the server may use the file attributes to - * store the verifier, the spec requires us to do a - * SETATTR RPC. FreeBSD servers store the verifier in - * atime, but we can't really assume that all servers - * will so we ensure that our SETATTR sets both atime - * and mtime. - */ - if (vap->va_mtime.tv_sec == VNOVAL) - vfs_timestamp(&vap->va_mtime); - if (vap->va_atime.tv_sec == VNOVAL) - vap->va_atime = vap->va_mtime; error = nfsrpc_setattr(newvp, vap, NULL, cnp->cn_cred, cnp->cn_thread, &nfsva, &attrflag, NULL); if (error && (vap->va_uid != (uid_t)VNOVAL || |
