Page MenuHomeFreeBSD
Differential D47148

nscd step6
Needs ReviewPublic
Actions

Authored by david_crossfamilyweb.com on Oct 16 2024, 3:10 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 29, 6:06 AM
Unknown Object (File)
Tue, Apr 28, 10:34 AM
Unknown Object (File)
Tue, Apr 21, 1:40 PM
Unknown Object (File)
Sun, Apr 19, 12:25 PM
Unknown Object (File)
Sat, Apr 11, 4:25 AM
Unknown Object (File)
Fri, Apr 10, 11:48 PM
Unknown Object (File)
Fri, Apr 10, 4:30 AM
Unknown Object (File)
Mon, Apr 6, 11:26 PM
View All 70 Files
Subscribers
imp

Details

Reviewers
markj
Summary

minor code quality fixups for getpwent.
implement libc side of grouplist and pivot lookups for caching

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

david_crossfamilyweb.com created this revision.Oct 16 2024, 3:10 AM
Herald added a subscriber: imp. · View Herald TranscriptOct 16 2024, 3:10 AM
david_crossfamilyweb.com requested review of this revision.Oct 16 2024, 3:10 AM
markj added inline comments.Oct 17 2024, 3:02 PM
lib/libc/gen/getgrent.c
209
370

original_ngroups is an "untrusted" value, i.e., it comes from nscd. This code should at least verify that

  1. it's non-negative,
  2. that original_ngroups * sizeof(gid_t) won't overflow,

before doing math with it.

david_crossfamilyweb.com updated this revision to Diff 145071.Oct 18 2024, 5:44 AM

Implemented all PR changes

david_crossfamilyweb.com mentioned this in D38047: Multiple fixes to the NSS caching system.Aug 6 2025, 11:02 PM

Revision Contents
Changeset List

PathSize
lib/
libc/
gen/
120 lines
6 lines
include/
3 lines

Diff 145071

View Options

lib/libc/gen/getgrent.c

Loading...
View Options

lib/libc/gen/getpwent.c

Loading...
View Options

lib/libc/include/nscachedcli.h

Loading...