Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | Fix generating configuration file | Baptiste Daroussin | 2015-06-07 | 1 | -4/+3 | |
| | ||||||
* | Fix duplicate checking | Baptiste Daroussin | 2015-06-07 | 1 | -10/+9 | |
| | ||||||
* | Remove uneeded code (already done by pw_make_v7) | Baptiste Daroussin | 2015-06-07 | 1 | -3/+0 | |
| | ||||||
* | In case of rename validate the length of the new name | Baptiste Daroussin | 2015-06-07 | 1 | -4/+4 | |
| | | | | Check early that the new name fits MAXLOGNAME and store it in pwconf | |||||
* | Refactor input validation | Baptiste Daroussin | 2015-06-07 | 1 | -58/+37 | |
| | | | | | Mutualize code to validate inputs of both 'user' and 'group' command Test that the input name fits into MAXLOGNAME | |||||
* | Finish converting -7 to pwconf | Baptiste Daroussin | 2015-06-07 | 1 | -4/+3 | |
| | ||||||
* | Handle -7 via gloval pwconf | Baptiste Daroussin | 2015-06-07 | 1 | -6/+5 | |
| | ||||||
* | Handle pretty print (-P) via global pwconf | Baptiste Daroussin | 2015-06-07 | 1 | -12/+6 | |
| | ||||||
* | Handle dryrun (-N) via global pwconf | Baptiste Daroussin | 2015-06-07 | 1 | -5/+3 | |
| | ||||||
* | Add a new global struct pwconf to store etcpath, rootdir and struct userconf | Baptiste Daroussin | 2015-06-07 | 1 | -14/+16 | |
| | | | | | Do not add anymore -R and -V to arglist Add an error message if both -V and -R are set in arguments | |||||
* | Fix pw userdel -r not deleting homedir | Baptiste Daroussin | 2015-06-07 | 1 | -1/+1 | |
| | ||||||
* | Test explicitly the result of strcmp to be consistent with the rest of the code | Baptiste Daroussin | 2015-06-07 | 1 | -1/+1 | |
| | ||||||
* | Improve readability by reducing indentations levels | Baptiste Daroussin | 2015-06-07 | 1 | -9/+11 | |
| | ||||||
* | Remove one level of indentation | Baptiste Daroussin | 2015-06-07 | 1 | -7/+6 | |
| | ||||||
* | Move user deletion code into a separate function to improve readability | Baptiste Daroussin | 2015-06-07 | 1 | -106/+115 | |
| | ||||||
* | Instead of always casting the pw_checkname input to u_char * and casting it back | Baptiste Daroussin | 2015-06-07 | 1 | -8/+9 | |
| | | | | to char *, change pw_checkname to directly take char * in input | |||||
* | New pw -R rootdir option | Baptiste Daroussin | 2015-06-03 | 1 | -11/+33 | |
| | | | | | | | | This allows to set an alternate root directory in which the users/groups will be manipulated Requested by: gjb, ian Tested by: gjb | |||||
* | Remove useless cast in printf and printf-like functions: | Baptiste Daroussin | 2015-05-31 | 1 | -15/+15 | |
| | | | | use %u for uid_t and gid_t | |||||
* | Cast -1 to uid_t and bump WARNING level to 3 | Baptiste Daroussin | 2015-05-31 | 1 | -2/+2 | |
| | ||||||
* | Make pw_user()/pw_group() more consitent about errors | Baptiste Daroussin | 2015-05-31 | 1 | -25/+15 | |
| | | | | | | | | 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) | |||||
* | Replace sprintf(3) with snprintf(3) | Baptiste Daroussin | 2015-05-09 | 1 | -7/+7 | |
| | ||||||
* | Use snprintf(3) instead of sprintf(3) | Baptiste Daroussin | 2015-05-09 | 1 | -8/+7 | |
| | | | | Remove useless "else" | |||||
* | Remove some uneeded headers | Baptiste Daroussin | 2015-05-09 | 1 | -1/+0 | |
| | ||||||
* | Replace malloc(3) + strcpy(3) + strcat(3) by asprintf(3) | Baptiste Daroussin | 2015-05-09 | 1 | -3/+1 | |
| | ||||||
* | Use snprintf(3) instead of strcpy(3) + strncat(3) | Baptiste Daroussin | 2015-05-09 | 1 | -2/+1 | |
| | ||||||
* | Revert r277652 | Baptiste Daroussin | 2015-01-26 | 1 | -4/+1 | |
| | | | | | | 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) | |||||
* | Allow negative numbers in -u and -g options | Baptiste Daroussin | 2015-01-24 | 1 | -1/+4 | |
| | | | | | PR: 196514 MFC after: 1 week | |||||
* | Ensure pw userdel -u <invalid> do not try to remove root | Baptiste Daroussin | 2014-10-28 | 1 | -0/+3 | |
| | | | | | | Check the uid passed is actually a number as early as possible MFC after: 1 week | |||||
* | Fix a regression in pw usermod -G list | Baptiste Daroussin | 2014-10-28 | 1 | -1/+19 | |
| | | | | | | | | | | The user was perperly adding the to different groups from "list" but was not removed from the other groups it could have belong to. While here add a regression test about this bug PR: 185666 Reported by: sub.mesa@gmail.com MFC after: 1 week | |||||
* | Two more places where login_setcryptfmt() defaults to MD5 were missed | Dag-Erling Smørgrav | 2014-10-09 | 1 | -2/+2 | |
| | | | | | | in r252688. MFC after: 3 days | |||||
* | pw: fix up deletion of users from groups | Mateusz Guzik | 2014-06-27 | 1 | -4/+3 | |
| | | | | | | | | | Previuosly given 'foo,bar' members, removing 'foo' would result in an infinite loop. PR: 191427 Submitted by: Voradesh Yenbut <yenbut cs.washington.edu> MFC after: 1 week | |||||
* | Fix pw(8) deletion of group "username" on userdel even if group "username" | Devin Teske | 2014-03-13 | 1 | -1/+9 | |
| | | | | | | | | | | | | is not associated with user "username". E.g., user "foo" has primary group "wheel" and is unassociated with group "foo", yet userdel would delete the group "foo" when deleting user "foo" (despite the fact that user "foo" is not associated with group "foo" in any way). Patch committed with minor style(9) changes. PR: bin/169471 Submitted by: Alexander Pyhalov <apyhalov@gmail.com> | |||||
* | Part 2 of bug 187310.. had to commit separately due to local confusion. | Julian Elischer | 2014-03-06 | 1 | -15/+21 | |
| | | | | | | | | Don't let pw crash when give certain input. PR: 187310 Submitted by: Kim Shrier MFC after: 1 week | |||||
* | Fall back to sha512 if passwd_format is not set. | Dag-Erling Smørgrav | 2013-07-04 | 1 | -1/+1 | |
| | | | | MFC after: 3 days | |||||
* | There is no reason to disallow setting the password or account expiry | Dag-Erling Smørgrav | 2013-07-03 | 1 | -4/+0 | |
| | | | | | | date to the current date. MFC after: 3 days | |||||
* | Fix -Wunsequenced warning | Tim Kientzle | 2013-06-29 | 1 | -1/+1 | |
| | | | | Submitted by: dt71@gmx.com | |||||
* | Prevent a null pointer dereference in pw userdel when deleting | Daniel Eischen | 2013-02-01 | 1 | -1/+1 | |
| | | | | a user whose group != username. | |||||
* | pw: free group returned by gr_add | Mateusz Guzik | 2013-01-06 | 1 | -0/+1 | |
| | ||||||
* | Simplify the code by using the new gr_add function | Baptiste Daroussin | 2012-12-27 | 1 | -15/+9 | |
| | ||||||
* | Simplify copying of group members by using memcpy | Baptiste Daroussin | 2012-12-26 | 1 | -2/+1 | |
| | | | | Submitted by: Christoph Mallon <christoph.mallon@gmx.de> | |||||
* | Fix off-by-one error in memory allocation: j entries, one new and a null | Baptiste Daroussin | 2012-12-26 | 1 | -1/+1 | |
| | | | | | | terminator is j + 2. Submitted by: Christoph Mallon <christoph.mallon@gmx.de> | |||||
* | Fix creating a user and adding it to a group | Baptiste Daroussin | 2012-12-26 | 1 | -7/+8 | |
| | | | | Reported by: "Sam Fourman Jr." <sfourman@gmail.com>, dim | |||||
* | Avoid overflow of file buffer | Eitan Adler | 2012-12-05 | 1 | -1/+1 | |
| | | | | | | Submitted by: db Approved by: cperciva MFC after: 2 weeks | |||||
* | Teach pw(8) about how to use pw/gr API to reduce code duplication | Baptiste Daroussin | 2012-10-30 | 1 | -7/+51 | |
| | | | | MFC after: 2 months | |||||
* | Do not treat empty name as an uid 0 | Baptiste Daroussin | 2012-10-01 | 1 | -1/+2 | |
| | | | | Reported by: Robert Bonomi <bonomi@mail.r-bonomi.com> | |||||
* | Remove useless atoi(3), previous strspn(3) makes sure that a_name->val is a | Baptiste Daroussin | 2012-10-01 | 1 | -2/+1 | |
| | | | | | | | | | number. This also allow pw user show to work as expected. PR: bin/172112 Submitted by: "Ilya A. Arkhipov" <rum1cro@yandex.ru> MFC after: 1 month | |||||
* | Handle NULL return from crypt(3). Mostly from DragonFly | Kevin Lo | 2012-02-22 | 1 | -1/+5 | |
| | ||||||
* | In usr.sbin/pw/pw_user.c, use the correct printf length modifier for a | Dimitry Andric | 2011-12-17 | 1 | -1/+1 | |
| | | | | | | ptrdiff_t. MFC after: 1 week | |||||
* | Backout r223115 which potentially caused a POLA violation, by restoring | Xin LI | 2011-07-31 | 1 | -1/+1 | |
| | | | | | | | | historic behavior (create the default base directory in pw.conf) before I came up with a better fix for this. Requested by: nwhitehorn Approved by: re (kib) | |||||
* | Don't attempt to create the base directory when -d is specified. | Xin LI | 2011-06-15 | 1 | -1/+1 | |
| | | | | MFC after: 1 month |