Page MenuHomeFreeBSD

malloc: Check for overflow when rounding up to the page size
Needs ReviewPublic

Authored by markj on Jul 24 2024, 9:31 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Feb 13, 9:56 AM
Unknown Object (File)
Wed, Feb 12, 3:42 AM
Unknown Object (File)
Tue, Jan 28, 2:38 PM
Unknown Object (File)
Dec 16 2024, 3:33 PM
Unknown Object (File)
Nov 29 2024, 3:47 AM
Unknown Object (File)
Nov 20 2024, 7:36 AM
Unknown Object (File)
Nov 5 2024, 3:21 PM
Unknown Object (File)
Sep 21 2024, 5:19 AM
Subscribers

Details

Reviewers
dougm

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 58808
Build 55695: arc lint + arc unit

Event Timeline

markj requested review of this revision.Jul 24 2024, 9:31 PM
sys/kern/kern_malloc.c
1140

Just return size.

1141

If PAGE_SHIFT >= KMEM_ZSHIFT (which is 4), then updating size with KMEM_ZMASK and ZMEM_ZBASE is a no-op. And I imagine that you can assume PAGE_SHIFT >= 4. So you can drop 2 lines here.

1141

This condition is now always false, since PAGE_SHIFT >= KMEM_ZSHIFT.

sys/kern/kern_malloc.c
612

This change uses _size and size for the unaligned and aligned size values. The next patch uses size and asize for those values. Consistency would be good.