Page MenuHomeFreeBSD
Differential D48471

fusefs: add more tests for buggy FUSE servers
ClosedPublic
Actions

Authored by asomers on Jan 15 2025, 8:22 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 4, 11:40 PM
Unknown Object (File)
Wed, Oct 29, 2:12 PM
Unknown Object (File)
Wed, Oct 29, 12:07 PM
Unknown Object (File)
Wed, Oct 29, 12:01 PM
Unknown Object (File)
Wed, Oct 29, 11:59 AM
Unknown Object (File)
Wed, Oct 29, 10:57 AM
Unknown Object (File)
Wed, Oct 29, 10:54 AM
Unknown Object (File)
Sun, Oct 26, 5:03 PM
View All 83 Files
Subscribers
imp

Details

Reviewers
rmacklem
Commits
rG5a8d8ffd7c8d: fusefs: add more checks for buggy FUSE servers
rGf1ec3bc06ed2: fusefs: add more checks for buggy FUSE servers
Summary
  • If a FUSE file system is NFS-exported (either by a kernel or userspace NFS server), then it must support FUSE_LOOKUP operations for ".". But if the response reports a different nodeid than the request, that's very bad. Fail the operation and warn the operator.
  • In general, a FUSE file may have a distinct "nodeid" and "inode number". But it the file system is NFS-exported (either by a kernel or userspace NFS server), then those two must match, because the NFS server will do VFS_VGET operations using the inode number. If they don't match, warn the operator.

MFC after: 2 weeks
Sponsored by: ConnectWise

Test Plan

Test cases added

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 61742
Build 58626: arc lint + arc unit

Revision Contents
Changeset List

PathSize
sys/
fs/
fuse/
2 lines
19 lines
15 lines
tests/
sys/
fs/
fusefs/
5 lines
7 lines
106 lines

Diff 149308

View Options

sys/fs/fuse/fuse_ipc.h

Loading...
View Options

sys/fs/fuse/fuse_node.c

Loading...
View Options

sys/fs/fuse/fuse_vfsops.c

Loading...
View Options

tests/sys/fs/fusefs/last_local_modify.cc

Loading...
View Options

tests/sys/fs/fusefs/lookup.cc

Loading...
View Options

tests/sys/fs/fusefs/nfs.cc

Loading...