diff options
author | Rick Macklem <rmacklem@FreeBSD.org> | 2022-08-26 03:33:31 +0000 |
---|---|---|
committer | Rick Macklem <rmacklem@FreeBSD.org> | 2022-08-26 03:33:31 +0000 |
commit | 2b612c9d3bb528551de9eaabbdbadae89a36ba8b (patch) | |
tree | 755376abec4da438ff13b5ee6e170ea45aee79c2 | |
parent | c1a0ab5ec525c4cb2dee4aad72d27f195e21450c (diff) | |
download | src-2b612c9d3bb528551de9eaabbdbadae89a36ba8b.tar.gz src-2b612c9d3bb528551de9eaabbdbadae89a36ba8b.zip |
nfscl: Fix handling of a bad session slot (NFSv4.1/4.2)
When a session has been marked defunct by the server
sending a NFSERR_BADSESSION reply to the NFSv4.1/4.2
client, nfsv4_sequencelookup() returns NFSERR_BADSESSION
without actually assigning a session slot.
Without this patch, newnfs_request() would erroneously
free slot 0.
This could result in the slot being reused prematurely,
but most likely just generated a "freeing free slot!!"
console message.
This patch fixes the code to not do the erroneous
freeing of the slot for this case.
PR: 260011
MFC after: 1 week
-rw-r--r-- | sys/fs/nfs/nfs_commonkrpc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sys/fs/nfs/nfs_commonkrpc.c b/sys/fs/nfs/nfs_commonkrpc.c index 150d92414e7a..916d0182d3ee 100644 --- a/sys/fs/nfs/nfs_commonkrpc.c +++ b/sys/fs/nfs/nfs_commonkrpc.c @@ -1080,12 +1080,12 @@ tryagain: } slot = nd->nd_slotid; } + freeslot = slot; } else if (slot != 0) { printf("newnfs_request: Bad " "session slot=%d\n", slot); slot = 0; } - freeslot = slot; if (retseq != sep->nfsess_slotseq[slot]) printf("retseq diff 0x%x\n", retseq); |