diff options
author | rmacklem <rmacklem@FreeBSD.org> | 2013-12-24 00:48:39 +0000 |
---|---|---|
committer | rmacklem <rmacklem@FreeBSD.org> | 2013-12-24 00:48:39 +0000 |
commit | 4c7a9b47dc3da58db48d2bbf450cc238a7b30950 (patch) | |
tree | b50fd5e0100ceb506d4e29ba1d70947f14cfcd47 /sys/fs/nfsclient | |
parent | 1c629a8317b4eec23f4fd1b14b9fca7a8062462f (diff) | |
download | FreeBSD-src-4c7a9b47dc3da58db48d2bbf450cc238a7b30950.zip FreeBSD-src-4c7a9b47dc3da58db48d2bbf450cc238a7b30950.tar.gz |
The NFSv4 client was passing both the p and cred arguments to
nfsv4_fillattr() as NULLs for the Getattr callback. This caused
nfsv4_fillattr() to not fill in the Change attribute for the reply.
I believe this was a violation of the RFC, but had little effect on
server behaviour. This patch passes a non-NULL p argument to fix this.
MFC after: 1 week
Diffstat (limited to 'sys/fs/nfsclient')
-rw-r--r-- | sys/fs/nfsclient/nfs_clstate.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sys/fs/nfsclient/nfs_clstate.c b/sys/fs/nfsclient/nfs_clstate.c index b378b8c..5fa302f 100644 --- a/sys/fs/nfsclient/nfs_clstate.c +++ b/sys/fs/nfsclient/nfs_clstate.c @@ -3279,7 +3279,7 @@ nfscl_docb(struct nfsrv_descript *nd, NFSPROC_T *p) FREE((caddr_t)nfhp, M_NFSFH); if (!error) (void) nfsv4_fillattr(nd, NULL, NULL, NULL, &va, - NULL, 0, &rattrbits, NULL, NULL, 0, 0, 0, 0, + NULL, 0, &rattrbits, NULL, p, 0, 0, 0, 0, (uint64_t)0); break; case NFSV4OP_CBRECALL: |