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
F166435396: D55111.diff
Thu, Aug 13, 1:47 PM
F166342103: D55111.id171166.diff
Thu, Aug 13, 12:39 AM
F166328348: D55111.id.diff
Wed, Aug 12, 10:52 PM
Unknown Object (File)
Mon, Aug 10, 2:01 PM
Unknown Object (File)
Mon, Aug 10, 12:05 PM
Unknown Object (File)
Mon, Aug 10, 11:55 AM
Unknown Object (File)
Mon, Aug 10, 8:10 AM
Unknown Object (File)
Sun, Aug 9, 2:58 AM
View All 76 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 Not Applicable
Unit
Tests Not Applicable

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 171221

View Options

usr.bin/diff/tests/diff_test.sh

Loading...