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
F168880815: D55111.id.diff
Sun, Aug 30, 2:23 PM
Unknown Object (File)
Sat, Aug 29, 2:42 AM
Unknown Object (File)
Thu, Aug 27, 2:56 AM
Unknown Object (File)
Tue, Aug 25, 8:04 PM
Unknown Object (File)
Tue, Aug 25, 12:00 AM
Unknown Object (File)
Fri, Aug 21, 5:24 AM
Unknown Object (File)
Thu, Aug 20, 4:25 AM
Unknown Object (File)
Thu, Aug 20, 4:25 AM
View All 86 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...