Paths

Table of Contentst

Differential D51046

scandir: Propagate errors from readdir().
ClosedPublic
Actions

Authored by des on Jun 25 2025, 8:11 PM.

Tags

None

Referenced Files

	F146267935: D51046.id.diff
	Sun, Mar 1, 6:28 AM

	F146253982: D51046.id157618.diff
	Sun, Mar 1, 3:46 AM

	Unknown Object (File)
	Sun, Feb 22, 9:08 PM

	Unknown Object (File)
	Mon, Feb 9, 3:14 PM

	Unknown Object (File)
	Fri, Feb 6, 7:44 AM

	Unknown Object (File)
	Dec 28 2025, 10:30 AM

	Unknown Object (File)
	Dec 28 2025, 2:25 AM

	Unknown Object (File)
	Dec 12 2025, 4:45 AM

View All 41 Files

Subscribers

Details

Reviewers

Group Reviewers

Commits

rG1a3c32bac0f0: scandir: Propagate errors from readdir().
rG62e0f12f5104: scandir: Propagate errors from readdir().

Summary

Currently, if readdir() fails, scandir() simply returns a partial
result (or a null result if it fails before any entries were selected).
There is no way within the current API design to return both a partial
result and an error indicator, so err on the side of caution: if an
error occurs, discard any partial result and return the error instead.

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.Jun 25 2025, 8:11 PM

Herald added subscribers: ziaee, imp. · View Herald TranscriptJun 25 2025, 8:11 PM

des requested review of this revision.Jun 25 2025, 8:11 PM

Harbormaster completed remote builds in B65087: Diff 157618.Jun 25 2025, 8:11 PM

markj accepted this revision as: markj.Jun 25 2025, 8:28 PM

This revision is now accepted and ready to land.Jun 25 2025, 8:28 PM

remove style changes

This revision now requires review to proceed.Jun 25 2025, 8:43 PM

Harbormaster completed remote builds in B65091: Diff 157623.Jun 25 2025, 8:43 PM

des added a child revision: D51050: scandir: Code cleanup..Jun 25 2025, 8:44 PM

markj accepted this revision as: markj.Jun 25 2025, 8:50 PM

This revision is now accepted and ready to land.Jun 25 2025, 8:50 PM

Closed by commit rG62e0f12f5104: scandir: Propagate errors from readdir(). (authored by des). · Explain WhyJun 26 2025, 7:38 AM

This revision was automatically updated to reflect the committed changes.

des added a commit: rG62e0f12f5104: scandir: Propagate errors from readdir()..

des added a commit: rG1a3c32bac0f0: scandir: Propagate errors from readdir()..Thu, Feb 5, 3:27 PM

Revision Contents
Changeset List

Path

Size

lib/

libc/

gen/

32 lines

12 lines

tests/

gen/

63 lines

Diff 157651

lib/libc/gen/scandir.3

Loading...

lib/libc/gen/scandir.c

Loading...

lib/libc/tests/gen/scandir_test.c

Loading...