system/freebsd-src
system/freebsd-src
1
0
Fork 0
mirror of https://github.com/freebsd/freebsd-src synced 2024-07-22 10:48:02 +00:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

setclassumask(): Accept 'inherit' as a value

Browse source 
'inherit' explicitly indicates that the umask should not be changed.

Reviewed by:            emaste
Approved by:            emaste (mentor)
MFC after:              3 days
Relnotes:               yes
Sponsored by:           Kumacom SAS
Differential Revision:  https://reviews.freebsd.org/D40687
This commit is contained in:
Olivier Certner 2023-06-20 21:41:04 +02:00 committed by Olivier Certner
parent 0dd1705f58
commit c328e6c6cc
No known key found for this signature in database
GPG key ID: 8CA13040971E2627
1 changed files with 12 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
lib/libutil/login_class.c
View file

@ -380,6 +380,11 @@ setclasscontext(const char *classname, unsigned int flags)
}
static const char * const inherit_enum[] = {
"inherit",
NULL
};
/*
* Private function setting umask from the login class.
*/
@ -391,7 +396,13 @@ setclassumask(login_cap_t *lc, const struct passwd *pwd)
* indicating no specification.
*/
const rlim_t def_val = INT64_MIN + 1, err_val = INT64_MIN;
const rlim_t val = login_getcapnum(lc, "umask", def_val, err_val);
rlim_t val;
/* If value is "inherit", nothing to change. */
if (login_getcapenum(lc, "umask", inherit_enum) == 0)
return;
val = login_getcapnum(lc, "umask", def_val, err_val);
if (val != def_val) {
if (val < 0 || val > UINT16_MAX) {