Page MenuHomeFreeBSD
Differential D45240

access(2): Discourage use of these system calls.
ClosedPublic
Actions

Authored by des on May 18 2024, 9:34 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, May 25, 11:04 PM
Unknown Object (File)
Wed, May 22, 7:34 PM
Unknown Object (File)
Wed, May 22, 4:48 PM
Unknown Object (File)
Wed, May 22, 3:54 PM
Unknown Object (File)
Wed, May 22, 12:28 PM
Unknown Object (File)
Tue, May 21, 10:36 PM
Unknown Object (File)
May 21 2024, 7:20 AM
Unknown Object (File)
May 20 2024, 8:52 PM
View All 9 Files
Subscribers
emaste
imp

Details

Reviewers
gbe
emaste
Group Reviewers
manpages
Commits
rGfb18c369c32d: access(2): Discourage use of these system calls.
rG55cd52a4418d: access(2): Discourage use of these system calls.
rG79b39f23e134: access(2): Discourage use of these system calls.
rGa4be1eb21165: access(2): Discourage use of these system calls.
Summary

Fixes: 421025a274fb
PR: 262895
MFC after: 3 days

Diff Detail

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

Event Timeline

des created this revision.May 18 2024, 9:34 AM
Herald added a subscriber: imp. · View Herald TranscriptMay 18 2024, 9:34 AM
des requested review of this revision.May 18 2024, 9:34 AM
Harbormaster completed remote builds in B57750: Diff 138711.May 18 2024, 9:35 AM
des updated this revision to Diff 138836.May 21 2024, 10:15 AM

reword

Harbormaster completed remote builds in B57814: Diff 138836.May 21 2024, 10:15 AM
emaste accepted this revision.EditedTue, May 21, 5:52 PM
emaste added a subscriber: emaste.

I think this is fine. Your comment in the PR that these answer the question what would happen if I tried to open() this path helps avoid any confusion that might exist. But I can't think of wording that is succinct, accurate, and covers all cases so don't have a suggestion to make.

This revision is now accepted and ready to land.Tue, May 21, 5:52 PM
imp added a comment.Tue, May 21, 6:35 PM

"Access and friends provide a good first guess if a future open might succeed." Might be good to insert at the start. Or "access, eaccess and faccessat provide..." and change the list at the start of the next sentence to "These system calls".

des retitled this revision from libsys: Remove misleading verbiage from access(2) to access(2): Discourage use of these system calls..Tue, May 21, 9:12 PM
des updated this revision to Diff 138862.Tue, May 21, 9:13 PM

be more explicit

This revision now requires review to proceed.Tue, May 21, 9:13 PM
Harbormaster completed remote builds in B57823: Diff 138862.Tue, May 21, 9:13 PM
emaste accepted this revision.Tue, May 21, 9:16 PM

LGTM. "whether an attempt to access the file" is definitely an improvement

This revision is now accepted and ready to land.Tue, May 21, 9:16 PM
Closed by commit rGa4be1eb21165: access(2): Discourage use of these system calls. (authored by des). · Explain WhyTue, May 21, 10:36 PM
This revision was automatically updated to reflect the committed changes.
des added a commit: rGa4be1eb21165: access(2): Discourage use of these system calls..
des added a commit: rG79b39f23e134: access(2): Discourage use of these system calls..Thu, May 23, 2:42 AM
des added a commit: rG55cd52a4418d: access(2): Discourage use of these system calls..Thu, May 23, 2:44 AM
des added a commit: rGfb18c369c32d: access(2): Discourage use of these system calls..Thu, May 23, 2:48 AM

Revision Contents
Changeset List

PathSize
lib/
libsys/
105 lines

Diff 138868

View Options

lib/libsys/access.2

Loading...