Page MenuHomeFreeBSD

kern: fail getgroup and setgroup with negative int
ClosedPublic

Authored by imp on Jun 2 2021, 6:14 PM.
Tags
None
Referenced Files
F121903533: D30617.id90338.diff
Mon, Jun 30, 12:23 PM
F121862139: D30617.id.diff
Mon, Jun 30, 2:51 AM
Unknown Object (File)
Sun, Jun 29, 2:37 AM
Unknown Object (File)
Sat, Jun 28, 8:02 AM
Unknown Object (File)
Tue, Jun 24, 1:20 AM
Unknown Object (File)
Sat, Jun 21, 2:13 PM
Unknown Object (File)
Thu, Jun 19, 4:23 PM
Unknown Object (File)
Wed, Jun 18, 12:28 PM
Subscribers
None

Details

Summary

Found using
https://github.com/NetBSD/src/blob/trunk/tests/lib/libc/sys/t_getgroups.c

getgroups/setgroups want an int and therefore casting it to u_int
resulted in getgroups(-1, ...) not returning -1 / errno = EINVAL.

imp@ updated syscall.master, without which this change would be a nop.

PR: 189941
Tested by: imp@

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

imp requested review of this revision.Jun 2 2021, 6:14 PM

Not in this review: fixing t_getgroups.c and the regenerated system calls headers.

markj added inline comments.
sys/kern/kern_prot.c
299

I would squash these into one line.

803

Same here.

This revision is now accepted and ready to land.Jun 2 2021, 6:19 PM
This revision was automatically updated to reflect the committed changes.