diff options
author | Rick Macklem <rmacklem@FreeBSD.org> | 2022-08-26 03:33:31 +0000 |
---|---|---|
committer | Rick Macklem <rmacklem@FreeBSD.org> | 2022-09-02 02:07:05 +0000 |
commit | 761033d3aace36eea1d82bbce86e06206b4cb17c (patch) | |
tree | 5a5f69c0f3e83c57f10fdc2e4661ceeacb12d1f8 | |
parent | a6f3979ad1923b7e742a6b4464917c9baea5b058 (diff) | |
download | src-761033d3aace36eea1d82bbce86e06206b4cb17c.tar.gz src-761033d3aace36eea1d82bbce86e06206b4cb17c.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
(cherry picked from commit 2b612c9d3bb528551de9eaabbdbadae89a36ba8b)
-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 81df586879c5..91ebd84fabd0 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); |