libc: nfsv4 acls: flip 'd' and 'D' in table definitions
Needs ReviewPublic
Actions

Authored by kevans on May 8 2025, 7:19 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Sep 21, 10:10 PM

	Unknown Object (File)
	Wed, Sep 17, 3:01 AM

	Unknown Object (File)
	Sep 13 2025, 5:44 PM

	Unknown Object (File)
	Sep 8 2025, 3:15 AM

	Unknown Object (File)
	Sep 6 2025, 8:10 AM

	Unknown Object (File)
	Sep 1 2025, 9:18 AM

	Unknown Object (File)
	Aug 31 2025, 6:57 AM

	Unknown Object (File)
	Aug 29 2025, 3:58 PM

View All 36 Files

Subscribers

imp

Details

Reviewers

emaste
trasz
walker.aj325_gmail.com

Summary

In theory this fixes the ordering for getfacl(1) to match what Illumos
and some other systems use, which has some benefits when trying to use
output from it in their tooling. FreeBSD's setfacl(1) doesn't seem to
do positional bits, so this change seems like it's both safe and likely
to improve interoperability.

PR: 286669
"Why not?": emaste

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 64014
Build 60898: arc lint + arc unit

Event Timeline

kevans created this revision.May 8 2025, 7:19 PM

Herald added a subscriber: imp. · View Herald TranscriptMay 8 2025, 7:19 PM

kevans requested review of this revision.May 8 2025, 7:19 PM

Harbormaster completed remote builds in B64014: Diff 155159.May 8 2025, 7:19 PM

Need to fix the title... should've mentioned NFSv4 ACLs. *sigh*

kevans retitled this revision from libc: flip 'd' and 'D' in nfs4 acl definitions to libc: nfsv4 acls: flip 'd' and 'D' in table definitions.May 8 2025, 7:20 PM

Revision Contents
Changeset List

Path

Size

lib/

libc/

posix1e/

acl_support_nfs4.c

2 lines

Diff 155159

View Options

lib/libc/posix1e/acl_support_nfs4.c

libc: nfsv4 acls: flip 'd' and 'D' in table definitionsNeeds ReviewPublicActions