Page MenuHomeFreeBSD
Differential D55111

diff: Add test case for pagination resource leak
ClosedPublic
Actions

Authored by des on Feb 4 2026, 6:35 PM.
Tags
None
Referenced Files
F151382967: D55111.id171166.diff
Wed, Apr 8, 1:12 AM
Unknown Object (File)
Sat, Mar 21, 5:49 AM
Unknown Object (File)
Sun, Mar 15, 5:18 AM
Unknown Object (File)
Sun, Mar 15, 5:18 AM
Unknown Object (File)
Mar 2 2026, 5:29 PM
Unknown Object (File)
Mar 2 2026, 1:41 AM
Unknown Object (File)
Feb 28 2026, 9:09 PM
Unknown Object (File)
Feb 27 2026, 3:42 PM
View All 17 Files
Subscribers
imp

Details

Reviewers
thj
kevans
Group Reviewers
Klara
Commits
rG2434f3b279a9: diff: Add test case for pagination resource leak
rGf0852209a086: diff: Add test case for pagination resource leak
rG51c2384ce049: diff: Add test case for pagination resource leak
rGa0a37c2143e8: diff: Add test case for pagination resource leak
rG270492602b9b: diff: Add test case for pagination resource leak
Summary

The pagination code leaks either processes or descriptors or both,
depending on the exact version of the code you have. Add a test case
which exercises this leak to facilitate fixing it.

MFC after: 1 week
Sponsored by: Klara, Inc.

Diff Detail

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

Event Timeline

des created this revision.Feb 4 2026, 6:35 PM
Herald added a subscriber: imp. · View Herald TranscriptFeb 4 2026, 6:35 PM
des requested review of this revision.Feb 4 2026, 6:35 PM
Harbormaster completed remote builds in B70410: Diff 171166.Feb 4 2026, 6:35 PM
des added a parent revision: D55110: diff: Fix integer overflows in Stone algorithm.Feb 4 2026, 6:35 PM
des added a child revision: D55112: diff: Fix pagination leak.
thj accepted this revision.Feb 5 2026, 1:03 PM
This revision is now accepted and ready to land.Feb 5 2026, 1:03 PM
kevans accepted this revision.Feb 5 2026, 2:23 PM
Closed by commit rG270492602b9b: diff: Add test case for pagination resource leak (authored by des). · Explain WhyFeb 5 2026, 2:46 PM
This revision was automatically updated to reflect the committed changes.
des added a commit: rG270492602b9b: diff: Add test case for pagination resource leak.
des added a commit: rGa0a37c2143e8: diff: Add test case for pagination resource leak.Feb 10 2026, 3:28 PM
des added a commit: rG51c2384ce049: diff: Add test case for pagination resource leak.
des added a commit: rGf0852209a086: diff: Add test case for pagination resource leak.Feb 10 2026, 3:33 PM
cperciva added a commit: rG2434f3b279a9: diff: Add test case for pagination resource leak.Feb 11 2026, 6:15 AM

Revision Contents
Changeset List

PathSize
usr.bin/
diff/
tests/
22 lines

Diff 171166

View Options

usr.bin/diff/tests/diff_test.sh

Loading...