Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | 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) | ||||
* | Fix building rcorder with -DDEBUG by using libutil.h instead of util.h from | Enji Cooper | 2015-01-25 | 0 | -0/+0 |
|\ | | | | | | | | | | | | | usr.bin/make MFC after: 1 week Sponsored by: EMC / Isilon Storage Division | ||||
| * | MFhead @ r277659 | Enji Cooper | 2015-01-24 | 5 | -3/+50 |
| |\ | |/ |/| | |||||
* | | Allow negative numbers in -u and -g options | Baptiste Daroussin | 2015-01-24 | 5 | -3/+50 |
| | | | | | | | | | | PR: 196514 MFC after: 1 week | ||||
| * | MFhead @ r277555 | Enji Cooper | 2015-01-23 | 0 | -0/+0 |
| |\ | |/ |/| | |||||
* | | Garbage collect a prove test wrapper | Enji Cooper | 2015-01-20 | 0 | -0/+0 |
|\| | | | | | | | | | MFC after: 3 days Sponsored by: EMC / Isilon Storage Division | ||||
| * | MFhead @ r276594 | Enji Cooper | 2015-01-02 | 0 | -0/+0 |
| |\ | |/ |/| | |||||
* | | Merge ^/head r274961 through r276472. | Dimitry Andric | 2014-12-31 | 0 | -0/+0 |
|\ \ | |||||
* | | | Merge ^/head r274961 through r276418. | Dimitry Andric | 2014-12-30 | 0 | -0/+0 |
|\| | | |||||
* | | | Merge ^/head r275759 through r275911. | Dimitry Andric | 2014-12-18 | 2 | -0/+89 |
|\| | | |||||
| | * | MFhead @ r275911 (also, sort out MK_* flags in BMAKE, etc on this branch) | Enji Cooper | 2014-12-18 | 2 | -0/+89 |
| | |\ | | |/ | |/| | |||||
| * | | 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 | ||||
| * | MFhead @ r275688 | Enji Cooper | 2014-12-10 | 0 | -0/+0 |
| |\ | |/ |/| | |||||
| * | MFhead @ r275663 | Enji Cooper | 2014-12-10 | 6 | -18/+135 |
| |\ | |/ |/| | |||||
* | | 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 |