| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
| |
Reported by: Coverity
MFC after: 1 week
Sponsored by: NetApp, Inc.
Sponsored by: Klara, Inc.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Mainly focus on files that use BSD 2-Clause license, however the tool I
was using misidentified many licenses so this was mostly a manual - error
prone - task.
The Software Package Data Exchange (SPDX) group provides a specification
to make it easier for automated tools to detect and summarize well known
opensource licenses. We are gradually adopting the specification, noting
that the tags are considered only advisory and do not, in any way,
superceed or replace the license texts.
No functional change intended.
|
|
|
|
|
|
| |
PR: 204676
Submitted by: longwitz@incore.de
MFC after: 2 days
|
|
|
|
|
| |
Reported by: Mike Selnet via forums.freebsd.org
MFC after: 3 days
|
| |
|
|
|
|
|
|
|
|
|
| |
by default
Reported by: Florian Degner <f.degner@gmx.de>
MFC after: 1 week
PR: 203876
Sponsored by: EMC / Isilon Storage Division
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Now each subcommands checks its arguments in a dedicated functions.
This helps improving input validation, code readability/maintainability
While here:
- Add a -y option to pw userdel/usermod so it can maintain NIS servers if
nispasswd is not defined in pw.conf(5)
- Allow pw -r <rootdir> to remove directory with userdel -r
- Fix bug when renaming a user which was not renaming the user name it groups
it is a member of.
- Only parse pw.conf(5) when needed.
|
|
|
|
| |
uid/gid size remains a implementation detail
|
| |
|
|
|
|
|
| |
Use gr_add(3) when possible to avoid code duplication.
Use a simpler logic to delete members of a group
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
of creating a user (regression from r285136)
Reported by: Fabian Keil <fk@fabiankeil.de>
|
| |
|
|
|
|
|
|
| |
While the return code is broken, some corner case usage depends on the
functionnality, so backout until we get better regression tests covering those
corner case usage.
|
| |
|
|
|
|
|
|
| |
the intent of -q in this command is to return as exit status the value of the
next group/user id, which does not make sense given exit status are limited to
values between 0 and 255.
|
| |
|
|
|
|
| |
Check early that the new name fits MAXLOGNAME and store it in pwconf
|
|
|
|
|
| |
Mutualize code to validate inputs of both 'user' and 'group' command
Test that the input name fits into MAXLOGNAME
|
| |
|
| |
|
|
|
|
|
| |
Do not add anymore -R and -V to arglist
Add an error message if both -V and -R are set in arguments
|
|
|
|
| |
to char *, change pw_checkname to directly take char * in input
|
| |
|
|
|
|
| |
use %u for uid_t and gid_t
|
| |
|
|
|
|
|
|
|
|
| |
Some of errors were returned to the main function, some others caused a direct
exit via err(3).
The main function is only interested in EXIT_SUCCESS, so in all other cases
replace warn(3) + return err by err(3)
|
|
|
|
|
|
| |
uid and gid are never and should never be negative. The pw(8) manpage clearly
states the -u and -g arguments are for uids/gids, hence using negative values is
abusing a bug in former versions of pw(8)
|
|
|
|
|
| |
PR: 196514
MFC after: 1 week
|
|
|
|
|
|
|
|
|
|
|
|
| |
Check that the -g argument is actually a number, if not report an error.
This argument is converted without checking with atoi(3) later so without this
check it converts any alpha entries into 0 meaning it deletes the group wheel
Add a regression test about it
PR: 90114
Reported by: bkoenig@cs.tu-berlin.de
MFC after: 1 week
|
|
|
|
|
|
|
|
|
| |
In that case get the group information using the new name.
Add a regression test about this bug
PR: 193704
Reported by: az
|
|
|
|
|
|
|
| |
PR:187310
Submitted by: Kim Shrier
Obtained from: bug
MFC after: 1 week
|
|
|
|
|
|
|
| |
print a newline after printing each group line.
PR: bin/174731
Submitted by: Jan Beich <jbeich@tormail.org>
|
|
|
|
|
| |
Approved by: cperciva
MFC after: 2 weeks
|
|
|
|
| |
MFC after: 2 months
|
|
|
|
|
|
|
|
|
|
| |
from a group without the need to perform the same operation by replacing
the existing list via the '-M' option. The '-M' option requires someone
to fetch the existing members with pw, deleting the undesired members from
the list and sending the altered list back to pw.
Approved by: wes (mentor)
MFC after: 5 days
|
|
|
|
|
| |
Approved by: wes (mentor)
MFC after: 5 days
|
|
|
|
|
|
|
| |
encrypted password on the specified file descriptor.
PR: bin/22033
MFC after: 2 weeks
|
|
|
|
|
| |
Use ISO 8601 date in logs.
Fix wrong argument type in ctype functions.
|
| |
|
|
|
|
|
|
| |
locking mechanism for users. This works by prepending the string
"*LOCKED*" to the password field in master.passwd to prevent successful
decoding.
|
|
|
|
|
|
|
|
|
|
|
|
| |
instead of a boolean. This replicated through he front-end sub-functions
relating to add, delete, modify entries in passwd & group files
Errno is now preserved so output of errc()/warnc() will be less obfuscated
by subsequent errors when reporting the problem.
Add more intelligent error handling when attempting to modify/delete NIS
entries with no corresponding local database entry.
[MFC to stable in a couple of weeks to keep both in sync]
|
| |
|