Page MenuHomeFreeBSD
Differential D51776

find: Fix -printf
ClosedPublic
Actions

Authored by des on Aug 6 2025, 10:16 PM.
Tags
None
Referenced Files
F134545294: D51776.id.diff
Mon, Nov 3, 2:05 AM
F134545283: D51776.id159925.diff
Mon, Nov 3, 2:05 AM
F134545116: D51776.id159937.diff
Mon, Nov 3, 2:03 AM
F134545110: D51776.id159927.diff
Mon, Nov 3, 2:03 AM
F134545063: D51776.id159937.diff
Mon, Nov 3, 2:03 AM
F134545043: D51776.id159927.diff
Mon, Nov 3, 2:03 AM
F134545039: D51776.id159932.diff
Mon, Nov 3, 2:03 AM
F134545035: D51776.id.diff
Mon, Nov 3, 2:03 AM
View All 37 Files
Subscribers
ziaee

Details

Reviewers
imp
Commits
rG02f394281fd6: find: Fix -printf
Summary

Each -printf invocation created a memstream, then freed the underlying
buffer without closing the stream, resulting in a segfault on exit
when libc tried to flush all the streams.

While here, simplify the implementation of %h.

Fixes: 7b9c912c41f4

Diff Detail

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

Event Timeline

des requested review of this revision.Aug 6 2025, 10:16 PM
des created this revision.
Harbormaster completed remote builds in B66044: Diff 159925.Aug 6 2025, 10:16 PM
des edited the summary of this revision. (Show Details)Aug 6 2025, 10:24 PM
des updated this revision to Diff 159927.Aug 6 2025, 10:24 PM
des edited the summary of this revision. (Show Details)

drop memstream

Harbormaster completed remote builds in B66046: Diff 159927.Aug 6 2025, 10:24 PM
des added a comment.Aug 6 2025, 10:24 PM

We should also try to figure out a way to call escape() only once.

imp accepted this revision.Aug 6 2025, 10:25 PM

This looks good. It matches what I have, plus fixes 'h'. So go ahead and test/commit it.

This revision is now accepted and ready to land.Aug 6 2025, 10:25 PM
des edited the summary of this revision. (Show Details)Aug 6 2025, 11:58 PM
des updated this revision to Diff 159932.Aug 6 2025, 11:58 PM
des edited the summary of this revision. (Show Details)

more

This revision now requires review to proceed.Aug 6 2025, 11:58 PM
Herald added a subscriber: ziaee. · View Herald TranscriptAug 6 2025, 11:58 PM
Harbormaster completed remote builds in B66048: Diff 159932.Aug 6 2025, 11:58 PM
imp accepted this revision.Aug 7 2025, 1:07 AM
This revision is now accepted and ready to land.Aug 7 2025, 1:07 AM
Closed by commit rG02f394281fd6: find: Fix -printf (authored by des). · Explain WhyAug 7 2025, 1:26 AM
This revision was automatically updated to reflect the committed changes.
des added a commit: rG02f394281fd6: find: Fix -printf.

Revision Contents
Changeset List

PathSize
usr.bin/
find/
13 lines
161 lines
tests/
111 lines

Diff 159932

View Options

usr.bin/find/find.1

Loading...
View Options

usr.bin/find/printf.c

Loading...
View Options

usr.bin/find/tests/find_test.sh

Loading...