Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | 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 | 3 | -11/+12 |
| | | | | to char *, change pw_checkname to directly take char * in input | ||||
* | Document pw(8) '-R' option, added in r283961. | Glen Barber | 2015-06-03 | 1 | -1/+20 |
| | | | | | | MFC after: 1 week X-MFC-after: r283961 Sponsored by: The FreeBSD Foundation | ||||
* | New pw -R rootdir option | Baptiste Daroussin | 2015-06-03 | 3 | -38/+89 |
| | | | | | | | | This allows to set an alternate root directory in which the users/groups will be manipulated Requested by: gjb, ian Tested by: gjb | ||||
* | Cast to gid_t for groups not uid_t | Baptiste Daroussin | 2015-05-31 | 1 | -1/+1 |
| | |||||
* | Remove useless cast in printf and printf-like functions: | Baptiste Daroussin | 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 | Baptiste Daroussin | 2015-05-31 | 3 | -4/+4 |
| | |||||
* | Move cleanup functions to the right place | Baptiste Daroussin | 2015-05-31 | 1 | -2/+3 |
| | |||||
* | Remove useless test before free() | Baptiste Daroussin | 2015-05-31 | 1 | -2/+1 |
| | |||||
* | Return early in case we cannot read the configuration file | Baptiste Daroussin | 2015-05-31 | 1 | -110/+111 |
| | | | | This drops one level of indentation | ||||
* | Make pw_user()/pw_group() more consitent about errors | Baptiste Daroussin | 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 | Baptiste Daroussin | 2015-05-31 | 1 | -4/+1 |
| | |||||
* | Remove uneeded intermediate variable | Baptiste Daroussin | 2015-05-31 | 1 | -4/+1 |
| | |||||
* | Use asprintf instead of malloc + snprintf and test the memory allocation | Baptiste Daroussin | 2015-05-31 | 1 | -2/+3 |
| | |||||
* | Remove dead declaration | Baptiste Daroussin | 2015-05-29 | 1 | -1/+0 |
| | | | | | PR: 195928 MFC after: 2 days | ||||
* | Remove extendline definition | Baptiste Daroussin | 2015-05-10 | 1 | -3/+0 |
| | | | | Remove now unused PWBUFSZ define | ||||
* | Use calloc(3) instead of malloc(3) + memset(3) | Baptiste Daroussin | 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 | Baptiste Daroussin | 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) | 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 |