Page MenuHomeFreeBSD
Differential D51680

kyua: Stop using readdir_r()
ClosedPublic
Actions

Authored by des on Aug 1 2025, 4:17 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Oct 26, 9:30 PM
Unknown Object (File)
Sun, Oct 26, 3:11 PM
Unknown Object (File)
Sat, Oct 25, 5:45 PM
Unknown Object (File)
Sat, Oct 18, 11:13 PM
Unknown Object (File)
Sat, Oct 18, 1:18 PM
Unknown Object (File)
Sat, Oct 18, 12:29 AM
Unknown Object (File)
Mon, Oct 13, 4:49 AM
Unknown Object (File)
Sat, Oct 11, 7:40 AM
View All 47 Files
Subscribers
emaste
igoro
imp

Details

Reviewers
igoro
Group Reviewers
Klara
Commits
rG805f6029286e: kyua: Stop using readdir_r()
rG927c3d709317: kyua: Stop using readdir_r()
rG65bae451c23b: kyua: Stop using readdir_r()
Summary

It cannot be used safely, and Kyua doesn't even pretend to try.

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.Aug 1 2025, 4:17 PM
Herald added a subscriber: imp. · View Herald TranscriptAug 1 2025, 4:17 PM
des requested review of this revision.Aug 1 2025, 4:17 PM
Harbormaster completed remote builds in B65897: Diff 159580.Aug 1 2025, 4:17 PM
des added a parent revision: D51679: libarchive: Stop using readdir_r().Aug 1 2025, 4:17 PM
des added a child revision: D51681: libc: Deprecate readdir_r().
des updated this revision to Diff 159606.Aug 1 2025, 10:35 PM

bogus free

Harbormaster completed remote builds in B65908: Diff 159606.Aug 1 2025, 10:35 PM
igoro accepted this revision.Aug 1 2025, 10:56 PM
igoro added a subscriber: igoro.

LGTM

contrib/kyua/utils/fs/directory.cpp
197

It seems it needs renaming to readdir not to let message readers think that readder_r was on the case.

This revision is now accepted and ready to land.Aug 1 2025, 10:56 PM
des marked an inline comment as done.Aug 1 2025, 11:08 PM
des updated this revision to Diff 159608.Aug 1 2025, 11:10 PM

exception

This revision now requires review to proceed.Aug 1 2025, 11:10 PM
Harbormaster completed remote builds in B65910: Diff 159608.Aug 1 2025, 11:10 PM
This revision was not accepted when it landed; it landed in state Needs Review.Aug 1 2025, 11:14 PM
Closed by commit rG65bae451c23b: kyua: Stop using readdir_r() (authored by des). · Explain Why
This revision was automatically updated to reflect the committed changes.
des added a commit: rG65bae451c23b: kyua: Stop using readdir_r().
emaste added a subscriber: emaste.Aug 1 2025, 11:25 PM

Also needs to be added to https://github.com/freebsd/kyua

des added a comment.Aug 1 2025, 11:53 PM
In D51680#1180679, @emaste wrote:

Also needs to be added to https://github.com/freebsd/kyua

See https://github.com/freebsd/kyua/commit/5d4df8e30bad13106c2ae58efad9bcb55659fc9e

des added a commit: rG927c3d709317: kyua: Stop using readdir_r().Aug 8 2025, 10:10 PM
des added a commit: rG805f6029286e: kyua: Stop using readdir_r().

Revision Contents
Changeset List

PathSize
contrib/
kyua/
utils/
fs/
18 lines

Diff 159611

View Options

contrib/kyua/utils/fs/directory.cpp

Loading...