summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | | | | New pw -R rootdir optionBaptiste Daroussin2015-06-033-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_tBaptiste Daroussin2015-05-311-1/+1
| | | | | | | | | |
* | | | | | | | | | Remove useless cast in printf and printf-like functions:Baptiste Daroussin2015-05-313-23/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | use %u for uid_t and gid_t
* | | | | | | | | | Cast -1 to uid_t and bump WARNING level to 3Baptiste Daroussin2015-05-313-4/+4
| | | | | | | | | |
* | | | | | | | | | Move cleanup functions to the right placeBaptiste Daroussin2015-05-311-2/+3
| | | | | | | | | |
* | | | | | | | | | Remove useless test before free()Baptiste Daroussin2015-05-311-2/+1
| | | | | | | | | |
* | | | | | | | | | Return early in case we cannot read the configuration fileBaptiste Daroussin2015-05-311-110/+111
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This drops one level of indentation
* | | | | | | | | | Make pw_user()/pw_group() more consitent about errorsBaptiste Daroussin2015-05-312-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 variableBaptiste Daroussin2015-05-311-4/+1
| | | | | | | | | |
* | | | | | | | | | Remove uneeded intermediate variableBaptiste Daroussin2015-05-311-4/+1
| | | | | | | | | |
* | | | | | | | | | Use asprintf instead of malloc + snprintf and test the memory allocationBaptiste Daroussin2015-05-311-2/+3
| | | | | | | | | |
* | | | | | | | | | Remove dead declarationBaptiste Daroussin2015-05-291-1/+0
| |_|_|_|_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | PR: 195928 MFC after: 2 days
* | | | | | | | | Remove extendline definitionBaptiste Daroussin2015-05-101-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove now unused PWBUFSZ define
* | | | | | | | | Use calloc(3) instead of malloc(3) + memset(3)Baptiste Daroussin2015-05-101-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | While here check the return of calloc(3)
* | | | | | | | | The initial logic for allocating the new string was wrong, the conversionBaptiste Daroussin2015-05-101-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 Daroussin2015-05-101-7/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Check the return of strndup
* | | | | | | | | Remove useless call to extendarrayBaptiste Daroussin2015-05-101-1/+0
| | | | | | | | |
* | | | | | | | | The pwdb function is only used once to check the database rename itBaptiste Daroussin2015-05-101-10/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | pwdb_check and simplify it accordingly
* | | | | | | | | if the check of the pw db fails return the failed valueBaptiste Daroussin2015-05-101-37/+35
| | | | | | | | |
* | | | | | | | | Some style(9) fixesBaptiste Daroussin2015-05-101-9/+16
| | | | | | | | |
* | | | | | | | | Replace sprintf(3) with snprintf(3)Baptiste Daroussin2015-05-091-7/+7
| | | | | | | | |
* | | | | | | | | Use snprintf(3) instead of sprintf(3)Baptiste Daroussin2015-05-091-8/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove useless "else"
* | | | | | | | | Remove now unused LNBUFSZ buffer sizeBaptiste Daroussin2015-05-091-3/+0
| | | | | | | | |
* | | | | | | | | Return from the function as early as possibleBaptiste Daroussin2015-05-091-117/+116
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reduces the depth of the if statements and improves clarity of the code
* | | | | | | | | Remove some uneeded headersBaptiste Daroussin2015-05-093-8/+0
| | | | | | | | |
* | | | | | | | | Remove unneeded headersBaptiste Daroussin2015-05-091-8/+0
| | | | | | | | |
* | | | | | | | | Replace malloc(3) + strcpy(3) + strcat(3) by asprintf(3)Baptiste Daroussin2015-05-091-3/+1
| | | | | | | | |
* | | | | | | | | Use snprintf(3) instead of strcpy(3) + strncat(3)Baptiste Daroussin2015-05-091-2/+1
| | | | | | | | |
* | | | | | | | | Use sbuf(9) instead of homebrewed buffered stringBaptiste Daroussin2015-05-093-62/+47
| | | | | | | | |
* | | | | | | | | Fix a race condition by allowing up to 5 seconds of difference between the ↵Brad Davis2015-02-241-4/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | time stamps. PR: 197861 Approved by: will
* | | | | | | | | Add tests for account and password expiration.Brad Davis2015-02-171-0/+111
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Approved by: will
* | | | | | | | | Really fix the pw_usernext tests.Brad Davis2015-02-151-12/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | PR: 197612 Approved by: will
* | | | | | | | | Remove an extra curly bracket that was causing intermittent failures.Brad Davis2015-02-141-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | PR: 197612 Submitted by: Robert O'Niel <oneil.rs@gmail.com> Approved by: will
* | | | | | | | | Add tests for `pw usernext'.Brad Davis2015-02-092-1/+44
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | PR: 197120 Submitted by: Robert O'Neil <oneil.rs@gmail.com> Approved by: will
* | | | | | | | | Revert r277652Baptiste Daroussin2015-01-265-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 fromEnji Cooper2015-01-250-0/+0
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | usr.bin/make MFC after: 1 week Sponsored by: EMC / Isilon Storage Division
| * \ \ \ \ \ \ \ \ MFhead @ r277659Enji Cooper2015-01-245-3/+50
| |\ \ \ \ \ \ \ \ \ | |/ / / / / / / / / |/| | | | | | | | |
* | | | | | | | | | Allow negative numbers in -u and -g optionsBaptiste Daroussin2015-01-245-3/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | PR: 196514 MFC after: 1 week
| * | | | | | | | | MFhead @ r277555Enji Cooper2015-01-230-0/+0
| |\ \ \ \ \ \ \ \ \ | |/ / / / / / / / / |/| | | | | | | | |
* | | | | | | | | | Garbage collect a prove test wrapperEnji Cooper2015-01-200-0/+0
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | MFC after: 3 days Sponsored by: EMC / Isilon Storage Division
| * | | | | | | | | MFhead @ r276594Enji Cooper2015-01-020-0/+0
| |\ \ \ \ \ \ \ \ \ | |/ / / / / / / / / |/| | | | | | | | |
* | | | | | | | | | Merge ^/head r274961 through r276472.Dimitry Andric2014-12-310-0/+0
|\ \ \ \ \ \ \ \ \ \
* | | | | | | | | | | Merge ^/head r274961 through r276418.Dimitry Andric2014-12-300-0/+0
|\| | | | | | | | | |
* | | | | | | | | | | Merge ^/head r275759 through r275911.Dimitry Andric2014-12-182-0/+89
|\| | | | | | | | | |
| | * | | | | | | | | MFhead @ r275911 (also, sort out MK_* flags in BMAKE, etc on this branch)Enji Cooper2014-12-182-0/+89
| | |\ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / | |/| | | | | | | | |
| * | | | | | | | | | Add tests for pw -NBrad Davis2014-12-162-0/+89
|/ / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | PR: 150449 Submitted by: Robert O'Neil <oneil.rs@gmail.com> Approved by: will
| * | | | | | | | | MFhead @ r275688Enji Cooper2014-12-100-0/+0
| |\ \ \ \ \ \ \ \ \ | |/ / / / / / / / / |/| | | | | | | | |
| * | | | | | | | | MFhead @ r275663Enji Cooper2014-12-106-18/+135
| |\ \ \ \ \ \ \ \ \ | |/ / / / / / / / / |/| | | | | | | | |
* | | | | | | | | | Restructure to make it easier to extend in the futureBrad Davis2014-12-091-6/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reviewed by: will Suggested by: ngie
* | | | | | | | | | Add some tests for user modification. [1]Brad Davis2014-12-092-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