Page MenuHomeFreeBSD

Control implicit PROT_MAX() using proccontrol(1) and the FreeBSD note feature bit.
ClosedPublic

Authored by kib on Jun 28 2019, 6:16 PM.
Tags
None
Referenced Files
Unknown Object (File)
Jan 30 2024, 7:19 AM
Unknown Object (File)
Dec 22 2023, 11:44 PM
Unknown Object (File)
Aug 2 2023, 12:07 AM
Unknown Object (File)
Aug 2 2023, 12:06 AM
Unknown Object (File)
Aug 2 2023, 12:04 AM
Unknown Object (File)
Aug 2 2023, 12:03 AM
Unknown Object (File)
Aug 1 2023, 11:55 PM
Unknown Object (File)
Jun 13 2023, 8:49 PM
Subscribers

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 25109

Event Timeline

Overall looks reasonable, a few small alpha ordering nits.

sys/compat/freebsd32/freebsd32_misc.c
3337

alpha order

sys/vm/vm_mmap.c
59

can this go in alpha order, other misordered headers notwithstanding?

usr.bin/proccontrol/proccontrol.c
76

alpha order

Suggested rewording of a sentence.

lib/libc/sys/procctl.2
129

The image flag might still prevent the enablement.

kib marked 4 inline comments as done.Jun 28 2019, 7:31 PM
kib added inline comments.
sys/vm/vm_mmap.c
59

Not sure what would be the right place then. Moved before filedesc.h.

lib/libc/sys/procctl.2
119

and .Xr mprotect 2

128

It would be nice to reference the sysctl here, "disabled system-wide by vm.imply_prot_max"

131

Should be "be disabled."

133

Extra _ between PROT and MAX.

136

"Use the system-wide ..."

kib marked an inline comment as done.

Man page update and minor code rearrangement.

This revision is now accepted and ready to land.Jul 2 2019, 3:00 PM
lib/libc/sys/procctl.2
119

We don't imply PROT_MAX on mprotect at this time. One could make a case for it, but we're never tested it and it's not as clear a case IMO.

sys/sys/syscallsubr.h
178

It seems a little odd to me that this is here and not in some sys/vm/* header, but don't have a strong opinion.