Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Use strndup(3) instead of malloc(3) + memcpy(3) | Baptiste Daroussin | 2015-05-10 | 1 | -7/+11 |
| | | | | Check the return of strndup | ||||
* | Remove useless call to extendarray | Baptiste Daroussin | 2015-05-10 | 1 | -1/+0 |
| | |||||
* | The pwdb function is only used once to check the database rename it | Baptiste Daroussin | 2015-05-10 | 1 | -10/+4 |
| | | | | pwdb_check and simplify it accordingly | ||||
* | if the check of the pw db fails return the failed value | Baptiste Daroussin | 2015-05-10 | 1 | -37/+35 |
| | |||||
* | Some style(9) fixes | Baptiste Daroussin | 2015-05-10 | 1 | -9/+16 |
| | |||||
* | 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 now unused LNBUFSZ buffer size | Baptiste Daroussin | 2015-05-09 | 1 | -3/+0 |
| | |||||
* | Return from the function as early as possible | Baptiste Daroussin | 2015-05-09 | 1 | -117/+116 |
| | | | | This reduces the depth of the if statements and improves clarity of the code | ||||
* | Remove some uneeded headers | Baptiste Daroussin | 2015-05-09 | 3 | -8/+0 |
| | |||||
* | Remove unneeded headers | Baptiste Daroussin | 2015-05-09 | 1 | -8/+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 |
| | |||||
* | Use sbuf(9) instead of homebrewed buffered string | Baptiste Daroussin | 2015-05-09 | 3 | -62/+47 |
| | |||||
* | Fix a race condition by allowing up to 5 seconds of difference between the ↵ | Brad Davis | 2015-02-24 | 1 | -4/+8 |
| | | | | | | | time stamps. PR: 197861 Approved by: will | ||||
* | Add tests for account and password expiration. | Brad Davis | 2015-02-17 | 1 | -0/+111 |
| | | | | Approved by: will | ||||
* | Really fix the pw_usernext tests. | Brad Davis | 2015-02-15 | 1 | -12/+15 |
| | | | | | PR: 197612 Approved by: will | ||||
* | Remove an extra curly bracket that was causing intermittent failures. | Brad Davis | 2015-02-14 | 1 | -1/+1 |
| | | | | | | PR: 197612 Submitted by: Robert O'Niel <oneil.rs@gmail.com> Approved by: will | ||||
* | Add tests for `pw usernext'. | Brad Davis | 2015-02-09 | 2 | -1/+44 |
| | | | | | | PR: 197120 Submitted by: Robert O'Neil <oneil.rs@gmail.com> Approved by: will | ||||
* | Revert r277652 | Baptiste Daroussin | 2015-01-26 | 5 | -50/+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) | ||||
* | Allow negative numbers in -u and -g options | Baptiste Daroussin | 2015-01-24 | 5 | -3/+50 |
| | | | | | PR: 196514 MFC after: 1 week | ||||
* | Add tests for pw -N | Brad Davis | 2014-12-16 | 2 | -0/+89 |
| | | | | | | PR: 150449 Submitted by: Robert O'Neil <oneil.rs@gmail.com> Approved by: will | ||||
* | Restructure to make it easier to extend in the future | Brad Davis | 2014-12-09 | 1 | -6/+10 |
| | | | | | Reviewed by: will Suggested by: ngie | ||||
* | Add some tests for user modification. [1] | Brad Davis | 2014-12-09 | 2 | -1/+59 |
| | | | | | | | | Fix a missing test in the Makefile from my previous commit. PR: 195834 [1] Submitted by: Robert O'Neil <oneil.rs@gmail.com> Approved by: will | ||||
* | Break out the tests into a file per command and clean up some long lines | Brad Davis | 2014-12-09 | 5 | -19/+33 |
| | | | | Approved by: will | ||||
* | Add more tests for pw(8) useradd. | Brad Davis | 2014-12-09 | 2 | -1/+42 |
| | | | | | | PR: 195832 Submitted by: Robert O'Neil <oneil.rs@gmail.com> Approved by: will | ||||
* | Convert to LIBADD. | Brad Davis | 2014-11-24 | 1 | -2/+1 |
| | | | | Reviewed by: bapt | ||||
* | Replace the pw(1) calls with a variable, to make it easier to to change ↵ | Brad Davis | 2014-11-19 | 4 | -32/+35 |
| | | | | | | across all the tests as needed. Reviewed by: will | ||||
* | Clean up some language | Brad Davis | 2014-11-17 | 2 | -18/+18 |
| | | | | Reviewed by: will | ||||
* | Add a test for locking and unlocking user accounts | Brad Davis | 2014-11-15 | 2 | -1/+23 |
| | | | | | Submitted by: Robert O'Neil <oneil.rs@gmail.com> Reviewed by: will | ||||
* | Fix wrong message when using pw -V with a non existent directory | Baptiste Daroussin | 2014-11-12 | 3 | -1/+27 |
| | | | | | | | Add a regression test about it PR: 194971 Submitted by: Freddy DISSAUX <bugzilla@dsx.bsdsx.fr> | ||||
* | Require root when running the pw_delete and pw_modify test programs (pw(8) | Enji Cooper | 2014-11-03 | 1 | -0/+3 |
| | | | | requires it) | ||||
* | Integrate usr.sbin/useradd/t_useradd.sh from NetBSD into FreeBSD as pw_test | Enji Cooper | 2014-11-03 | 1 | -0/+11 |
| | | | | | | | | | - Suffix useradd/userdel commands with pw - Remove the atf_expect_fail for bin/39546 Phabric: D921 Reviewed by: brd Sponsored by: EMC / Isilon Storage Division | ||||
* | Fix renaming a group via the gr_copy function | Baptiste Daroussin | 2014-10-28 | 1 | -0/+14 |
| | | | | | | | | | | Add a regression test to pw(8) because the bug was discovered via using: pw groupmod PR: 187189 Reported by: mcdouga9@egr.msu.edu Tested by: mcdouga9@egr.msu.edu Patch by: Marc de la Gueronniere | ||||
* | Ensure pw userdel -u <invalid> do not try to remove root | Baptiste Daroussin | 2014-10-28 | 2 | -0/+14 |
| | | | | | | Check the uid passed is actually a number as early as possible MFC after: 1 week | ||||
* | Fix atf syntax | Baptiste Daroussin | 2014-10-28 | 1 | -1/+1 |
| | |||||
* | Do not delete the group wheel when bad argument is passed to pw groupdel -g | Baptiste Daroussin | 2014-10-28 | 2 | -0/+17 |
| | | | | | | | | | | | | 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 | ||||
* | Fix a regression in pw usermod -G list | Baptiste Daroussin | 2014-10-28 | 2 | -1/+40 |
| | | | | | | | | | | 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 | ||||
* | When a group is renamed then the group has been invalidated for sure. | Baptiste Daroussin | 2014-10-28 | 2 | -3/+17 |
| | | | | | | | | | In that case get the group information using the new name. Add a regression test about this bug PR: 193704 Reported by: az | ||||
* | 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 | ||||
* | - Add some tests for modifying groups | Brad Davis | 2014-10-06 | 2 | -1/+35 |
| | | | | Reviewed by: will | ||||
* | - Add a test for bug 191427 where pw(8) will go into an infinite loop | Brad Davis | 2014-10-02 | 6 | -0/+62 |
| | | | | | Reviewed by: will MFC after: 1 month | ||||
* | 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 changing the username | Baptiste Daroussin | 2014-06-20 | 1 | -1/+1 |
| | | | | | | PR: 189172 Submitted by: fullermd@over-yonder.net MFC after: 1 week | ||||
* | Removed compatibility with pre FreeBSD 2.2 pw_mkdb command | Baptiste Daroussin | 2014-06-20 | 1 | -13/+2 |
| | | | | | | | | Fix some broken indentattion PR: 189173 Submitted by: fullermd@over-yonder.net MFC after: 1 week | ||||
* | Simplify reading pw.conf(5) by using getline(3) | Baptiste Daroussin | 2014-04-22 | 2 | -24/+12 |
| | |||||
* | 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 | ||||
* | Stop pw(8) from segfaulting when given certain input | Julian Elischer | 2014-03-06 | 1 | -6/+13 |
| | | | | | | | PR:187310 Submitted by: Kim Shrier Obtained from: bug 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 |