diff options
| author | brooks <brooks@FreeBSD.org> | 2010-02-24 15:52:18 +0000 |
|---|---|---|
| committer | brooks <brooks@FreeBSD.org> | 2010-02-24 15:52:18 +0000 |
| commit | e7e1754f546de4b9a2a6525196bad4f98030e416 (patch) | |
| tree | 0411c38d3342498748016567cf33336a548fddee /sys/kern/subr_param.c | |
| parent | b557f570b2b71d4ecf8371b3c9a574c75e6938d8 (diff) | |
| download | FreeBSD-src-e7e1754f546de4b9a2a6525196bad4f98030e416.zip FreeBSD-src-e7e1754f546de4b9a2a6525196bad4f98030e416.tar.gz | |
Don't inforce an upper bound on kern.ngroups. The INT_MAX-1 limit was
too high due to several overflows. The actual limit is somewhere in the
neighborhood of INT_MAX/4 on 64-bit machines, but most systems could not
support such a limit due to a lack of memory and the cost of duplicate
credentials.
Reported by: bde
Diffstat (limited to 'sys/kern/subr_param.c')
| -rw-r--r-- | sys/kern/subr_param.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/sys/kern/subr_param.c b/sys/kern/subr_param.c index fcd8131..1504a78 100644 --- a/sys/kern/subr_param.c +++ b/sys/kern/subr_param.c @@ -240,8 +240,6 @@ init_param1(void) TUNABLE_INT_FETCH("kern.ngroups", &ngroups_max); if (ngroups_max < NGROUPS_MAX) ngroups_max = NGROUPS_MAX; - if (ngroups_max > INT_MAX - 1) - ngroups_max = INT_MAX - 1; } /* |
