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)
Sat, Sep 12, 2:24 PM
Unknown Object (File)
Thu, Sep 10, 10:49 PM
Unknown Object (File)
Wed, Sep 9, 8:48 PM
Unknown Object (File)
Sat, Sep 5, 8:30 PM
Unknown Object (File)
Sat, Sep 5, 2:55 PM
Unknown Object (File)
Fri, Sep 4, 5:44 PM
Unknown Object (File)
Thu, Sep 3, 10:48 PM
Unknown Object (File)
Mon, Aug 24, 7:14 AM
View All 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...