Page MenuHomeFreeBSD
Differential D54673

linuxkpi: Clean up linux_shmem_file_setup() a bit
ClosedPublic
Actions

Authored by markj on Jan 12 2026, 6:56 PM.
Tags
None
Referenced Files
F162895205: D54673.id.diff
Sat, Jul 18, 2:33 AM
Unknown Object (File)
Wed, Jul 15, 7:02 AM
Unknown Object (File)
Wed, Jul 15, 7:02 AM
Unknown Object (File)
Sat, Jul 4, 12:00 PM
Unknown Object (File)
Jun 18 2026, 9:28 AM
Unknown Object (File)
Jun 17 2026, 2:23 AM
Unknown Object (File)
Jun 12 2026, 12:36 PM
Unknown Object (File)
May 7 2026, 11:29 PM
View All 51 Files
Subscribers
brooks
emaste
imp

Details

Reviewers
emaste
bz
brooks
Group Reviewers
linuxkpi
cheri
Commits
rGf1e1bfe228e9: linuxkpi: Clean up linux_shmem_file_setup() a bit
rG251662e5abdd: linuxkpi: Clean up linux_shmem_file_setup() a bit
Summary
  • Free the pointer that was returned by the allocator, instead of the address of the first member. These will be equal in practice, but it's sketchy and possibly won't work on CHERI.
  • Use an anonymous struct, there's no need to name it.

Diff Detail

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

Event Timeline

markj created this revision.Jan 12 2026, 6:56 PM
Herald added subscribers: emaste, imp. · View Herald TranscriptJan 12 2026, 6:56 PM
markj requested review of this revision.Jan 12 2026, 6:56 PM
Herald added a reviewer: cheri. · View Herald TranscriptJan 12 2026, 6:56 PM
markj added a parent revision: D54672: linuxkpi: Avoid a potential null pointer dereference in an error path.Jan 12 2026, 6:56 PM
Harbormaster completed remote builds in B69831: Diff 169535.Jan 14 2026, 12:50 AM
emaste accepted this revision.Jan 14 2026, 4:26 PM
This revision is now accepted and ready to land.Jan 14 2026, 4:26 PM
bz accepted this revision.Jan 14 2026, 5:02 PM
brooks accepted this revision.Jan 15 2026, 11:38 AM
brooks added a subscriber: brooks.

In practice, this would be fine with default CHERI C, but broken with subobject bounds.

Closed by commit rG251662e5abdd: linuxkpi: Clean up linux_shmem_file_setup() a bit (authored by markj). · Explain WhyJan 15 2026, 2:20 PM
This revision was automatically updated to reflect the committed changes.
markj added a commit: rG251662e5abdd: linuxkpi: Clean up linux_shmem_file_setup() a bit.
markj added a commit: rGf1e1bfe228e9: linuxkpi: Clean up linux_shmem_file_setup() a bit.Jan 26 2026, 12:53 AM

Revision Contents
Changeset List

PathSize
sys/
compat/
linuxkpi/
common/
src/
7 lines

Diff 169777

View Options

sys/compat/linuxkpi/common/src/linux_shmemfs.c

Loading...