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
Unknown Object (File)
Sat, Sep 19, 10:14 PM
Unknown Object (File)
Thu, Sep 17, 10:44 PM
Unknown Object (File)
Sun, Sep 13, 5:05 AM
Unknown Object (File)
Sat, Sep 12, 11:18 AM
Unknown Object (File)
Fri, Sep 11, 9:41 PM
Unknown Object (File)
Tue, Sep 8, 5:34 PM
Unknown Object (File)
Tue, Sep 8, 5:31 PM
Unknown Object (File)
Tue, Sep 8, 2:29 AM
View All 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...