Page MenuHomeFreeBSD

tmpfs: push VEXEC check into tmpfs_lookup()
ClosedPublic

Authored by kevans on Jan 28 2021, 3:49 PM.
Tags
None
Referenced Files
F122274646: D28401.id83081.diff
Thu, Jul 3, 11:16 PM
Unknown Object (File)
Wed, Jul 2, 12:09 AM
Unknown Object (File)
Thu, Jun 26, 4:47 PM
Unknown Object (File)
Sun, Jun 22, 2:47 PM
Unknown Object (File)
Wed, Jun 18, 11:24 PM
Unknown Object (File)
Sat, Jun 14, 7:07 PM
Unknown Object (File)
Wed, Jun 11, 12:40 PM
Unknown Object (File)
Tue, Jun 10, 2:51 AM
Subscribers

Details

Summary

vfs_cache_lookup() has already done the appropriate VEXEC check, therefore
we must not re-check in VOP_CACHEDLOOKUP.

This fixes O_SEARCH semantics on tmpfs and removes a redundant descent into
VOP_ACCESS() in the common case.

Reported-by: arichardson (via CheriBSD Jenkins CI)
Differential-Revision: https://reviews.freebsd.org/D28401

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 36590
Build 33479: arc lint + arc unit

Event Timeline

Maybe Reported-by: arichardson (via CheriBSD Jenkins CI)

sys/fs/tmpfs/tmpfs_vnops.c
102

Instead of adding cached param, you can move vn_dir_check_exec() into tmpfs_lookup().

Just push the check back into tmpfs_lookup, note in tmpfs_lookup1 that the
caller should have ensured it's a valid access to make sure it's obvious to
readers that it's intentionally omitted from here.

kevans retitled this revision from tmpfs: do not recheck for VEXEC in VOP_CACHEDLOOKUP to tmpfs: push VEXEC check into tmpfs_lookup().Jan 28 2021, 10:40 PM
kevans edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Jan 28 2021, 11:03 PM
This revision was automatically updated to reflect the committed changes.