Page MenuHomeFreeBSD
Differential D28987

add safetybelt checks for cache_enter_time() when an NFS server sends a bogus reply
ClosedPublic
Actions

Authored by rmacklem on Feb 28 2021, 3:55 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jan 24, 4:42 PM
Unknown Object (File)
Thu, Jan 9, 7:03 PM
Unknown Object (File)
Thu, Jan 9, 8:02 AM
Unknown Object (File)
Dec 26 2024, 3:08 AM
Unknown Object (File)
Dec 25 2024, 9:25 PM
Unknown Object (File)
Dec 25 2024, 3:03 PM
Unknown Object (File)
Dec 25 2024, 9:45 AM
Unknown Object (File)
Dec 10 2024, 12:37 PM
View All 75 Files
Subscribers
None

Details

Reviewers
kib
mjg
Commits
rG3e04ab36ba5c: nfsclient: add checks for a server returning the current directory
Summary

D28969 dealt with a panic in cache_enter_time() where
the vnode referred to the directory.
It would also be possible to get these panics if a broken
NFS server were to return the directory as an new object being
created.

This patch adds checks to avoid the panics and logs
messages to indicate that the server is broken.

Test Plan

Ran connectathon test suite for various mount options.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

rmacklem requested review of this revision.Feb 28 2021, 3:55 PM
rmacklem created this revision.
kib accepted this revision.Feb 28 2021, 6:42 PM
This revision is now accepted and ready to land.Feb 28 2021, 6:42 PM
Closed by commit rG3e04ab36ba5c: nfsclient: add checks for a server returning the current directory (authored by rmacklem). · Explain WhyFeb 28 2021, 10:17 PM
This revision was automatically updated to reflect the committed changes.
rmacklem added a commit: rG3e04ab36ba5c: nfsclient: add checks for a server returning the current directory.

Revision Contents
Changeset List

PathSize
sys/
fs/
nfsclient/
38 lines

Diff 84877

View Options

sys/fs/nfsclient/nfs_clvnops.c

Loading...