Page MenuHomeFreeBSD

mprotect.2: Improve the description of prot
ClosedPublic

Authored by brooks on Sep 1 2021, 9:28 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Dec 28, 7:56 AM
Unknown Object (File)
Oct 14 2024, 9:49 PM
Unknown Object (File)
Oct 2 2024, 3:01 PM
Unknown Object (File)
Sep 29 2024, 3:57 PM
Unknown Object (File)
Sep 2 2024, 7:55 AM
Unknown Object (File)
Sep 1 2024, 9:35 PM
Unknown Object (File)
Aug 16 2024, 1:45 AM
Unknown Object (File)
Aug 7 2024, 6:28 PM
Subscribers

Details

Summary

The new wording is based on the POSIX description.

Sponsored by: DARPA

Diff Detail

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

Event Timeline

As requested in D30442, this improves the wording of the description of the prot argument.

The wording is much better, and I don't think changes the intended meaning...

This revision is now accepted and ready to land.Sep 1 2021, 9:33 PM
lib/libc/sys/mprotect.2
56

Technically, the argument should be the bitwise OR of some subset of {READ, WRITE, EXEC} or PROT_NONE. ORing PROT_NONE with other flags does nothing.

  • Separate PROT_NONE from other arguments
This revision now requires review to proceed.Sep 2 2021, 6:47 PM
This revision is now accepted and ready to land.Sep 2 2021, 9:09 PM
markj added inline comments.
lib/libc/sys/mprotect.2
54
lib/libc/sys/mprotect.2
73

While we're here, I would suggest adding the word "later" here, i.e., saying, "... from later upgrading ..." Also, wouldn't "adding" be clearer than "upgrading"?

75

Add the word "bitwise" here, like above?

I agree with mark and alc comments.

  • Adopt reviewer suggestions
  • Render "or" consistently
This revision now requires review to proceed.Sep 2 2021, 11:53 PM
This revision was not accepted when it landed; it landed in state Needs Review.Sep 7 2021, 4:29 PM
This revision was automatically updated to reflect the committed changes.