Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Remove useless cast in printf and printf-like functions: | 2015-05-31 | 3 | -23/+23 | |
| | | | | use %u for uid_t and gid_t | ||||
* | Cast -1 to uid_t and bump WARNING level to 3 | 2015-05-31 | 3 | -4/+4 | |
| | |||||
* | Move cleanup functions to the right place | 2015-05-31 | 1 | -2/+3 | |
| | |||||
* | Remove useless test before free() | 2015-05-31 | 1 | -2/+1 | |
| | |||||
* | Return early in case we cannot read the configuration file | 2015-05-31 | 1 | -110/+111 | |
| | | | | This drops one level of indentation | ||||
* | Make pw_user()/pw_group() more consitent about errors | 2015-05-31 | 2 | -37/+24 | |
| | | | | | | | | 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) | ||||
* | Remove uneeded intermediate variable | 2015-05-31 | 1 | -4/+1 | |
| | |||||
* | Remove uneeded intermediate variable | 2015-05-31 | 1 | -4/+1 | |
| | |||||
* | Use asprintf instead of malloc + snprintf and test the memory allocation | 2015-05-31 | 1 | -2/+3 | |
| | |||||
* | Remove dead declaration | 2015-05-29 | 1 | -1/+0 | |
| | | | | | PR: 195928 MFC after: 2 days | ||||
* | Remove extendline definition | 2015-05-10 | 1 | -3/+0 | |
| | | | | Remove now unused PWBUFSZ define | ||||
* | Use calloc(3) instead of malloc(3) + memset(3) | 2015-05-10 | 1 | -2/+4 | |
| | | | | While here check the return of calloc(3) | ||||
* | The initial logic for allocating the new string was wrong, the conversion | 2015-05-10 | 1 | -5/+2 | |
| | | | | | to strndup(3) duplicated the same mistake, actually strdup(3) is good enough to allocate the new string. | ||||
* | Use strndup(3) instead of malloc(3) + memcpy(3) | 2015-05-10 | 1 | -7/+11 | |
| | | | | Check the return of strndup | ||||
* | Remove useless call to extendarray | 2015-05-10 | 1 | -1/+0 | |
| | |||||
* | The pwdb function is only used once to check the database rename it | 2015-05-10 | 1 | -10/+4 | |
| | | | | pwdb_check and simplify it accordingly | ||||
* | if the check of the pw db fails return the failed value | 2015-05-10 | 1 | -37/+35 | |
| | |||||
* | Some style(9) fixes | 2015-05-10 | 1 | -9/+16 | |
| | |||||
* | Replace sprintf(3) with snprintf(3) | 2015-05-09 | 1 | -7/+7 | |
| | |||||
* | Use snprintf(3) instead of sprintf(3) | 2015-05-09 | 1 | -8/+7 | |
| | | | | Remove useless "else" | ||||
* | Remove now unused LNBUFSZ buffer size | 2015-05-09 | 1 | -3/+0 | |
| | |||||
* | Return from the function as early as possible | 2015-05-09 | 1 | -117/+116 | |
| | | | | This reduces the depth of the if statements and improves clarity of the code | ||||
* | Remove some uneeded headers | 2015-05-09 | 3 | -8/+0 | |
| | |||||
* | Remove unneeded headers | 2015-05-09 | 1 | -8/+0 | |
| | |||||
* | Replace malloc(3) + strcpy(3) + strcat(3) by asprintf(3) | 2015-05-09 | 1 | -3/+1 | |
| | |||||
* | Use snprintf(3) instead of strcpy(3) + strncat(3) | 2015-05-09 | 1 | -2/+1 | |
| | |||||
* | Use sbuf(9) instead of homebrewed buffered string | 2015-05-09 | 3 | -62/+47 | |
| | |||||
* | Fix a race condition by allowing up to 5 seconds of difference between the ↵ | 2015-02-24 | 1 | -4/+8 | |
| | | | | | | | time stamps. PR: 197861 Approved by: will | ||||
* | Add tests for account and password expiration. | 2015-02-17 | 1 | -0/+111 | |
| | | | | Approved by: will | ||||
* | Really fix the pw_usernext tests. | 2015-02-15 | 1 | -12/+15 | |
| | | | | | PR: 197612 Approved by: will | ||||
* | Remove an extra curly bracket that was causing intermittent failures. | 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'. | 2015-02-09 | 2 | -1/+44 | |
| | | | | | | PR: 197120 Submitted by: Robert O'Neil <oneil.rs@gmail.com> Approved by: will | ||||
* | Revert r277652 | 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 | 2015-01-24 | 5 | -3/+50 | |
| | | | | | PR: 196514 MFC after: 1 week | ||||
* | Add tests for pw -N | 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 | 2014-12-09 | 1 | -6/+10 | |
| | | | | | Reviewed by: will Suggested by: ngie | ||||
* | Add some tests for user modification. [1] | 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 | 2014-12-09 | 5 | -19/+33 | |
| | | | | Approved by: will | ||||
* | Add more tests for pw(8) useradd. | 2014-12-09 | 2 | -1/+42 | |
| | | | | | | PR: 195832 Submitted by: Robert O'Neil <oneil.rs@gmail.com> Approved by: will | ||||
* | Convert to LIBADD. | 2014-11-24 | 1 | -2/+1 | |
| | | | | Reviewed by: bapt | ||||
* | Replace the pw(1) calls with a variable, to make it easier to to change ↵ | 2014-11-19 | 4 | -32/+35 | |
| | | | | | | across all the tests as needed. Reviewed by: will | ||||
* | Clean up some language | 2014-11-17 | 2 | -18/+18 | |
| | | | | Reviewed by: will | ||||
* | Add a test for locking and unlocking user accounts | 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 | 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) | 2014-11-03 | 1 | -0/+3 | |
| | | | | requires it) | ||||
* | Integrate usr.sbin/useradd/t_useradd.sh from NetBSD into FreeBSD as pw_test | 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 | 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 | 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 | 2014-10-28 | 1 | -1/+1 | |
| | |||||
* | Do not delete the group wheel when bad argument is passed to pw groupdel -g | 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 |