Page MenuHomeFreeBSD
Differential D48471

fusefs: add more tests for buggy FUSE servers
Needs ReviewPublic
Actions

Authored by asomers on Jan 15 2025, 8:22 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Feb 14, 6:57 PM
Unknown Object (File)
Wed, Feb 5, 5:20 PM
Unknown Object (File)
Tue, Feb 4, 10:59 PM
Unknown Object (File)
Fri, Jan 31, 5:17 AM
Unknown Object (File)
Thu, Jan 30, 2:30 AM
Unknown Object (File)
Wed, Jan 29, 8:23 PM
Unknown Object (File)
Wed, Jan 29, 7:01 PM
Unknown Object (File)
Fri, Jan 24, 3:28 PM
View All 18 Files
Subscribers
imp

Details

Reviewers
rmacklem
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 61743
Build 58627: arc lint + arc unit

Revision Contents
Changeset List

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

Diff 149309

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...