Differential D47013

nmount(2), NFS: Accept 'ngroups_max + 1' groups in "export" credentials
ClosedPublic
Actions

Authored by olce on Oct 8 2024, 1:47 PM.

Tags

None

Referenced Files

	F168800852: D47013.diff
	Sun, Aug 30, 5:40 AM

	Unknown Object (File)
	Sat, Aug 29, 1:15 AM

	Unknown Object (File)
	Thu, Aug 27, 1:37 AM

	Unknown Object (File)
	Mon, Aug 24, 7:32 PM

	Unknown Object (File)
	Mon, Aug 24, 7:14 PM

	Unknown Object (File)
	Tue, Aug 18, 4:29 PM

	Unknown Object (File)
	Mon, Aug 17, 7:02 PM

	Unknown Object (File)
	Sun, Aug 16, 8:31 PM

Subscribers

Details

Reviewers

Commits

rG46f0ef933d28: nmount(2), NFS: Accept 'ngroups_max + 1' groups in "export" credentials
rGd8b77cc05ab2: nmount(2), NFS: Accept 'ngroups_max + 1' groups in "export" credentials
rG84337218786b: nmount(2), NFS: Accept 'ngroups_max + 1' groups in "export" credentials

Summary

There is no technical reason to limit that to NGROUPS_MAX, which is off
by one and just our minimum maximum value.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

olce created this revision.Oct 8 2024, 1:47 PM

Herald added a subscriber: imp. · View Herald TranscriptOct 8 2024, 1:47 PM

olce requested review of this revision.Oct 8 2024, 1:47 PM

Harbormaster completed remote builds in B59844: Diff 144488.Oct 8 2024, 1:47 PM

olce added a parent revision: D46921: mountd(8): parsecred(): Groups limit: NGROUPS_MAX => NGROUPS_MAX + 1.Oct 8 2024, 1:47 PM

olce added a child revision: D47014: mountd(8): parsecred(): Remove comment on non-existent bug.Oct 8 2024, 1:48 PM

olce mentioned this in D46920: mountd(8): parsecred(): Remove "duplicate compression".Oct 8 2024, 2:43 PM

Looks fine to me, since ngroups_max can never
be set < NGROUPS_MAX.

This revision is now accepted and ready to land.Oct 9 2024, 2:29 AM

olce mentioned this in D46921: mountd(8): parsecred(): Groups limit: NGROUPS_MAX => NGROUPS_MAX + 1.Oct 14 2024, 3:11 PM

Closed by commit rG84337218786b: nmount(2), NFS: Accept 'ngroups_max + 1' groups in "export" credentials (authored by olce). · Explain WhyDec 16 2024, 2:46 PM

This revision was automatically updated to reflect the committed changes.

olce added a commit: rG84337218786b: nmount(2), NFS: Accept 'ngroups_max + 1' groups in "export" credentials.

olce added a commit: rGd8b77cc05ab2: nmount(2), NFS: Accept 'ngroups_max + 1' groups in "export" credentials.Jan 16 2025, 6:09 PM

olce added a commit: rG46f0ef933d28: nmount(2), NFS: Accept 'ngroups_max + 1' groups in "export" credentials.Jan 17 2025, 12:28 PM

Revision Contents
Changeset List

Path

Size

sys/

kern/

2 lines

Diff 148040

sys/kern/vfs_mount.c

Loading...