Page MenuHomeFreeBSD

kern.ipc.{msqids,semsegs,sema} sysctls for FreeBSD32.
ClosedPublic

Authored by brooks on Dec 12 2017, 2:33 AM.
Tags
None
Referenced Files
Unknown Object (File)
Nov 15 2024, 10:10 AM
Unknown Object (File)
Nov 1 2024, 8:05 PM
Unknown Object (File)
Oct 10 2024, 5:18 PM
Unknown Object (File)
Sep 22 2024, 9:51 PM
Unknown Object (File)
Sep 16 2024, 12:29 PM
Unknown Object (File)
Sep 11 2024, 3:40 PM
Unknown Object (File)
Sep 7 2024, 2:42 PM
Unknown Object (File)
Sep 6 2024, 1:26 AM

Details

Summary

Stop leaking kernel pointers though theses sysctls and make sure that the
padding in the structures is zeroed on allocation to avoid other leaks.

Sponsored by: DARPA, AFRL
Obtained from: CheriBSD

Test Plan

needs testing with 32-bit ipcs or the like on a 64-bit system

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This is technically an information disclosure, but in practice is quite small and unlikely to be a serious issue in the current world order.

gordon added a subscriber: gordon.

Looks okay to me but I'm probably not the best person to judge. If anyone else would like to weigh in. Feel free.

This revision is now accepted and ready to land.Dec 15 2017, 6:15 AM
kib added a subscriber: kib.
kib added inline comments.
sys/compat/freebsd32/freebsd32_ipc.h
53 ↗(On Diff #36486)

Wny _kernel32 and not e.g. compat32 ?

sys/kern/sysv_msg.c
232 ↗(On Diff #36486)

Spaces around '|'.

sys/compat/freebsd32/freebsd32_ipc.h
53 ↗(On Diff #36486)

Because this is the 32-bit version of struct semid_kernel.

sys/kern/sysv_msg.c
232 ↗(On Diff #36486)

Will fix shortly.

This revision now requires review to proceed.Jan 30 2018, 5:59 PM
This revision was not accepted when it landed; it landed in state Needs Review.Feb 2 2018, 6:03 PM
This revision was automatically updated to reflect the committed changes.