Page MenuHomeFreeBSD
Differential D36959

linuxkpi: Add `krealloc_array()`
ClosedPublic
Actions

Authored by dumbbell on Oct 12 2022, 7:17 PM.
Tags
None
Referenced Files
F166819927: D36959.id112654.diff
Sun, Aug 16, 8:56 PM
Unknown Object (File)
Sun, Aug 16, 3:01 AM
Unknown Object (File)
Sat, Aug 15, 7:49 PM
Unknown Object (File)
Fri, Aug 14, 6:39 PM
Unknown Object (File)
Thu, Aug 13, 7:17 PM
Unknown Object (File)
Thu, Aug 13, 5:20 PM
Unknown Object (File)
Thu, Aug 13, 5:20 PM
Unknown Object (File)
Thu, Aug 13, 5:19 PM
View All Files
Subscribers
emaste
• hselasky
imp
linuxkpi

Details

Reviewers
manu
Commits
rGe1534bafdab9: linuxkpi: Add `krealloc_array()`
rG1ad6b2b1daa8: linuxkpi: Add `krealloc_array()`
Summary

In FreeBSD, this is a wrapper on top of realloc().

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

dumbbell created this revision.Oct 12 2022, 7:17 PM
Herald added subscribers: linuxkpi, emaste, imp. · View Herald TranscriptOct 12 2022, 7:17 PM
dumbbell requested review of this revision.Oct 12 2022, 7:17 PM
Harbormaster completed remote builds in B47807: Diff 111729.Oct 12 2022, 7:17 PM
dumbbell mentioned this in D36869: This patch contains [[ https://github.com/dumbbell/freebsd-src/tree/linuxkpi-changes-for-drm-from-5.11 | a series of commits to linuxkpi ]] in order to support an update of the DRM drivers to match Linux 5.11:.Oct 12 2022, 7:32 PM
dumbbell added a subscriber: • hselasky.Oct 12 2022, 7:35 PM
dumbbell added inline comments.
sys/compat/linuxkpi/common/include/linux/slab.h
184

In the original review (D36869), @hselasky said:

Are we missing any n * size overflow checks?

In a reply to his comment, I replied with the following questions:

What is the usual behavior to react to such overflow? Fail and return NULL? Panic?

If you know of an example in the kernel, could you please share it?

emaste added inline comments.Oct 12 2022, 7:43 PM
sys/compat/linuxkpi/common/include/linux/slab.h
184

Linux returns NULL on overflow.

dumbbell updated this revision to Diff 112654.Nov 5 2022, 1:44 PM

Thank you @hselasky and @emaste! I updated the patch to check overflow.

dumbbell marked an inline comment as done.Nov 5 2022, 2:02 PM
manu accepted this revision.Nov 5 2022, 5:19 PM
This revision is now accepted and ready to land.Nov 5 2022, 5:19 PM
Closed by commit rG1ad6b2b1daa8: linuxkpi: Add `krealloc_array()` (authored by dumbbell). · Explain WhyNov 11 2022, 5:59 PM
This revision was automatically updated to reflect the committed changes.
dumbbell added a commit: rG1ad6b2b1daa8: linuxkpi: Add `krealloc_array()`.
manu added a commit: rGe1534bafdab9: linuxkpi: Add `krealloc_array()`.Jan 24 2023, 8:56 AM

Revision Contents
Changeset List

PathSize
sys/
compat/
linuxkpi/
common/
include/
linux/
10 lines

Diff 112654

View Options

sys/compat/linuxkpi/common/include/linux/slab.h

Loading...