Paths

Table of Contentst

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

	F86906795: D45240.diff
	Thu, Jun 27, 7:36 AM

	F86854474: D45240.diff
	Wed, Jun 26, 2:55 PM

	Unknown Object (File)
	May 25 2024, 11:04 PM

	Unknown Object (File)
	May 22 2024, 7:34 PM

	Unknown Object (File)
	May 22 2024, 4:48 PM

	Unknown Object (File)
	May 22 2024, 3:54 PM

	Unknown Object (File)
	May 22 2024, 12:28 PM

	Unknown Object (File)
	May 21 2024, 10:36 PM

View All 11 Files

Subscribers

Details

Reviewers

gbe
emaste

Group Reviewers

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 Skipped

Unit

Tests Skipped

Build Status

Buildable 57814
Build 54702: arc lint + arc unit

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

reword

Harbormaster completed remote builds in B57814: Diff 138836.May 21 2024, 10:15 AM

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.May 21 2024, 5:52 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..May 21 2024, 9:12 PM

be more explicit

This revision now requires review to proceed.May 21 2024, 9:13 PM

Harbormaster completed remote builds in B57823: Diff 138862.May 21 2024, 9:13 PM

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

This revision is now accepted and ready to land.May 21 2024, 9:16 PM

Closed by commit rGa4be1eb21165: access(2): Discourage use of these system calls. (authored by des). · Explain WhyMay 21 2024, 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..May 23 2024, 2:42 AM

des added a commit: rG55cd52a4418d: access(2): Discourage use of these system calls..May 23 2024, 2:44 AM

des added a commit: rGfb18c369c32d: access(2): Discourage use of these system calls..May 23 2024, 2:48 AM

Revision Contents
Changeset List

Path

Size

lib/

libsys/

45 lines

Diff 138836

lib/libsys/access.2

Loading...