Page MenuHomeFreeBSD

fix getdirentries man page for NFS client returning d_off == 0
ClosedPublic

Authored by rmacklem on Feb 14 2021, 4:34 PM.
Tags
None
Referenced Files
F102417336: D28664.diff
Tue, Nov 12, 12:03 AM
Unknown Object (File)
Oct 6 2024, 4:05 AM
Unknown Object (File)
Oct 4 2024, 11:45 PM
Unknown Object (File)
Oct 3 2024, 4:52 AM
Unknown Object (File)
Oct 3 2024, 1:13 AM
Unknown Object (File)
Oct 1 2024, 2:49 PM
Unknown Object (File)
Oct 1 2024, 11:47 AM
Unknown Object (File)
Sep 27 2024, 4:41 PM
Subscribers
None

Details

Summary

PR3253428 reported that getdirentries(2) was
returning dirents with d_off set to 0 for an NFS
mount.

This is believed to be correct behaviour at
this time (it may change for some NFS mounts
in the future), but is inconsistent with what the
getdirentries(2) man page says.

This patch fixes the man page.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

rmacklem created this revision.
lib/libc/sys/getdirentries.2
95

I find this confusing. Is the antecedent of "it" "cookie", "directory descriptor", or "entry"? I would say "returns a cookie which, if non-zero, can be used with lseek to position the directory descriptor to the next entry."

151

I would shorten this to "if non-zero".

Apply changes suggested by asomers@.

Marked inline comments as done.

Approved, but you should fix the bug number in the commit message.

This revision is now accepted and ready to land.Feb 14 2021, 10:34 PM
This revision was automatically updated to reflect the committed changes.