summaryrefslogtreecommitdiffstats
path: root/pw
Commit message (Collapse)AuthorAgeFilesLines
* 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)
* Allow negative numbers in -u and -g optionsBaptiste Daroussin2015-01-245-3/+50
| | | | | PR: 196514 MFC after: 1 week
* Add tests for pw -NBrad Davis2014-12-162-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 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
* Break out the tests into a file per command and clean up some long linesBrad Davis2014-12-095-19/+33
| | | | Approved by: will
* Add more tests for pw(8) useradd.Brad Davis2014-12-092-1/+42
| | | | | | PR: 195832 Submitted by: Robert O'Neil <oneil.rs@gmail.com> Approved by: will
* Convert to LIBADD.Brad Davis2014-11-241-2/+1
| | | | Reviewed by: bapt
* Replace the pw(1) calls with a variable, to make it easier to to change ↵Brad Davis2014-11-194-32/+35
| | | | | | across all the tests as needed. Reviewed by: will
* Clean up some languageBrad Davis2014-11-172-18/+18
| | | | Reviewed by: will
* Add a test for locking and unlocking user accountsBrad Davis2014-11-152-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 directoryBaptiste Daroussin2014-11-123-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 Cooper2014-11-031-0/+3
| | | | requires it)
* Integrate usr.sbin/useradd/t_useradd.sh from NetBSD into FreeBSD as pw_testEnji Cooper2014-11-031-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 functionBaptiste Daroussin2014-10-281-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 rootBaptiste Daroussin2014-10-282-0/+14
| | | | | | Check the uid passed is actually a number as early as possible MFC after: 1 week
* Fix atf syntaxBaptiste Daroussin2014-10-281-1/+1
|
* Do not delete the group wheel when bad argument is passed to pw groupdel -gBaptiste Daroussin2014-10-282-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 listBaptiste Daroussin2014-10-282-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
generated by cgit v1.2.3-56-ge451 (git 2.39.0) at 2024-07-02 00:25:02 +0000