Paths

Table of Contentst

Differential D51046

scandir: Propagate errors from readdir().
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F122191451: D51046.id157623.diff
	Thu, Jul 3, 4:09 AM

	Unknown Object (File)
	Wed, Jul 2, 7:39 AM

	Unknown Object (File)
	Wed, Jul 2, 5:09 AM

	Unknown Object (File)
	Wed, Jul 2, 4:37 AM

	Unknown Object (File)
	Sun, Jun 29, 5:28 PM

	Unknown Object (File)
	Sat, Jun 28, 3:28 PM

	Unknown Object (File)
	Thu, Jun 26, 7:38 AM

Subscribers

Details

Reviewers

Group Reviewers

Commits

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 Skipped

Unit

Tests Skipped

Build Status

Buildable 65087
Build 61970: arc lint + arc unit

Event Timeline

des created this revision.Wed, Jun 25, 8:11 PM

Herald added subscribers: ziaee, imp. · View Herald TranscriptWed, Jun 25, 8:11 PM

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

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

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

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

remove style changes

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

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

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

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

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

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

This revision was automatically updated to reflect the committed changes.

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

Revision Contents
Changeset List

Path

Size

lib/

libc/

gen/

30 lines

20 lines

tests/

gen/

63 lines

Diff 157618

lib/libc/gen/scandir.3

Loading...

lib/libc/gen/scandir.c

Loading...

lib/libc/tests/gen/scandir_test.c

Loading...