summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Fix useradd regression:Baptiste Daroussin2015-08-211-0/+66
| | | | | | | | Readd the function to create the parents home directory if it does not exists. if it is only a directory at the top level of the hierarchy symlink it into /usr as it used to be done before. Reported by: kevlo, adrian
* Regression: fix pw usermod -w xxxBaptiste Daroussin2015-08-141-0/+2
| | | | Reported by: gjb
* MFP r276712.Alexander V. Chernikov2015-08-110-0/+0
|\ | | | | | | | | | | | | | | * Split lltable_init() into lltable_allocate_htbl() (alloc hash table with default callbacks) and lltable_link() ( links any lltable to the list). * Switch from LLTBL_HASHTBL_SIZE to per-lltable hash size field. * Move lltable setup to separate functions in in[6]_domifattach.
| * Sync to HEAD@r274766Alexander V. Chernikov2014-11-211-0/+8
| |\
* | | MFP r274553:Alexander V. Chernikov2015-08-080-0/+0
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Move lle creation/deletion from lla_lookup to separate functions: lla_lookup(LLE_CREATE) -> lla_create lla_lookup(LLE_DELETE) -> lla_delete lla_create now returns with LLE_EXCLUSIVE lock for lle. * Provide typedefs for new/existing lltable callbacks. Reviewed by: ae
| * | Sync to HEAD@r274095.Alexander V. Chernikov2014-11-043-9/+45
| |\ \
| * \ \ Merge HEAD@r272944.Alexander V. Chernikov2014-10-112-2/+8
| |\ \ \
* | | | | Avoid calling strlen() where we can use the strspn() return value.Ed Schouten2015-08-032-9/+9
| | | | |
* | | | | Clarify pw(8) manual w/respect to required arguments. Break long lines atDevin Teske2015-08-031-90/+118
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | punctuation while here. Differential Revision: https://reviews.freebsd.org/D2700 Reviewed by: wblock, bapt MFC after: 3 days X-MFC-to: stable/10
* | | | | Fix bugs spotted by gccBaptiste Daroussin2015-08-031-3/+4
| | | | | | | | | | | | | | | | | | | | Reported by: adrian
* | | | | Actually set quiet to something.Adrian Chadd2015-08-031-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | /usr/home/adrian/work/freebsd/head-embedded-2/src/usr.sbin/pw/pw_user.c: In function 'pw_user_next': /usr/home/adrian/work/freebsd/head-embedded-2/src/usr.sbin/pw/pw_user.c:680: warning: statement with no effect
* | | | | Use intmax_t rather than long longBaptiste Daroussin2015-08-021-2/+2
| | | | |
* | | | | Fix build on 32bitsBaptiste Daroussin2015-08-021-2/+2
| | | | |
* | | | | Split some extra long linesBaptiste Daroussin2015-08-021-2/+4
| | | | |
* | | | | Split some extra long linesBaptiste Daroussin2015-08-021-5/+10
| | | | |
* | | | | Cleanup a bit includesBaptiste Daroussin2015-08-0212-52/+43
| | | | |
* | | | | Fix regression: report if a group already exists when creating itBaptiste Daroussin2015-08-021-0/+2
| | | | |
* | | | | Fix regression: report again if a username already exists when creating itBaptiste Daroussin2015-08-021-0/+3
| | | | |
* | | | | Remove dead codeBaptiste Daroussin2015-08-021-11/+0
| | | | |
* | | | | Rewrite parsing subcommands arguments of pw(8)Baptiste Daroussin2015-08-0210-1482/+1951
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now each subcommands checks its arguments in a dedicated functions. This helps improving input validation, code readability/maintainability While here: - Add a -y option to pw userdel/usermod so it can maintain NIS servers if nispasswd is not defined in pw.conf(5) - Allow pw -r <rootdir> to remove directory with userdel -r - Fix bug when renaming a user which was not renaming the user name it groups it is a member of. - Only parse pw.conf(5) when needed.
* | | | | Remove things that crept in after badly checked revertBaptiste Daroussin2015-08-011-2/+0
| | | | |
* | | | | Partial revert of r286152Baptiste Daroussin2015-08-014-16/+6
| | | | | | | | | | | | | | | | | | | | More work needed on the cli validation
* | | | | Fix buildBaptiste Daroussin2015-08-011-0/+1
| | | | |
* | | | | Fix formatting of new codeBaptiste Daroussin2015-08-011-20/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix sorting or errstr Remove useless initialisation or errstr Reported by: bde
* | | | | Validate expiration days and password days from commmand line and pw.confBaptiste Daroussin2015-08-014-8/+28
| | | | |
* | | | | Validate the max_uid/max_gid boundaries and entry type in pw.confBaptiste Daroussin2015-08-011-7/+23
| | | | |
* | | | | Cast uid/git to uintmax_t when using printf-like functions so the size ofBaptiste Daroussin2015-08-013-23/+27
| | | | | | | | | | | | | | | | | | | | uid/gid size remains a implementation detail
* | | | | Improve strtounumBaptiste Daroussin2015-07-302-24/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix many style bugs Better variable naming Use C99 'restrict' were apropriate Fix potential errno race Submitted by: bde
* | | | | Cleanup includesBaptiste Daroussin2015-07-291-6/+3
| | | | |
* | | | | Actually set the proper licenseBaptiste Daroussin2015-07-291-12/+13
| | | | | | | | | | | | | | | | | | | | Reported by: trasz
* | | | | Actually add the new codeBaptiste Daroussin2015-07-291-0/+73
| | | | |
* | | | | Create a strtounum function using the same API as strtonumBaptiste Daroussin2015-07-293-4/+8
| | | | | | | | | | | | | | | | | | | | | | | | | This function returns uintmax_t Use this function to convert to gid_t/uid_t
* | | | | Reject usermod and userdel if the user concerned is not on the user databaseBaptiste Daroussin2015-07-281-0/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | supposed to be manipulated This prevent pw usermod creating a new local user when requesting to usermod on a username is defined in LDAP. This issue only happens when modifying the local user database (not inpacting commands when -V or -R are used). PR: 187653 Submitted by: tmwalaszek@gmail.com
* | | | | Check uid/gid used when creating a user/group are not larger than ↵Baptiste Daroussin2015-07-281-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | UID_MAX/GID_MAX PR: 173977 Reported by: nvass@gmx.com
* | | | | Fix wrong warning printed after changing or updating NIS usersBaptiste Daroussin2015-07-281-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | PR: 37672 Submitted by: chris+freebsd@chrullrich.de
* | | | | when -n is passed to any pw subcommand it is always expected to be considered asBaptiste Daroussin2015-07-281-8/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | a name so do not try to convert it to an id if it is a numeric value PR: 31933 Reported by: ted@impulse.net Sponsored by: gandi.net
* | | | | Really fix -oBaptiste Daroussin2015-07-131-1/+1
| | | | |
* | | | | Regression fix: allow to create users with uid0Baptiste Daroussin2015-07-131-1/+1
| | | | | | | | | | | | | | | | | | | | Reported by: Jan Mikkelsen <janm@transactionware.com>
* | | | | Fix logic of check duplicates that has been invertedBaptiste Daroussin2015-07-131-1/+2
| | | | |
* | | | | Ensure skeldir is abolute path (relatively to the rootdir)Baptiste Daroussin2015-07-121-4/+10
| | | | |
* | | | | pw -R <rootdir> userdel can now cleanup installationBaptiste Daroussin2015-07-123-84/+76
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rewrite rm_r to use *at function, allowing to remove home directories along with users. only crontabs and at(1) installation are not removed Relnotes: yes
* | | | | Rework the home directory creation and copy or the skel content to use *atBaptiste Daroussin2015-07-124-87/+90
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | functions This allows to simplify the code a bit for -R by not having to keep modifying path and also prepare the code to improve support -R in userdel While here, add regression tests for the functionality
* | | | | Make getarg return NULL if args is NULLBaptiste Daroussin2015-07-122-5/+7
| | | | |
* | | | | Fix regression: ensure when try to create the group and the user with the sameBaptiste Daroussin2015-07-111-12/+4
| | | | | | | | | | | | | | | | | | | | id if possible and nothing in particular was specified
* | | | | Remove now unused variableBaptiste Daroussin2015-07-111-3/+0
| | | | |
* | | | | Replace custom string array with stringlist(3)Baptiste Daroussin2015-07-115-95/+35
| | | | |
* | | | | Rework groupmod modification:Baptiste Daroussin2015-07-111-61/+29
| | | | | | | | | | | | | | | | | | | | | | | | | Use gr_add(3) when possible to avoid code duplication. Use a simpler logic to delete members of a group
* | | | | Remove unused argument from pm_passwdBaptiste Daroussin2015-07-111-4/+4
| | | | |
* | | | | check the gecos format early: at the moment the -c option is parsedBaptiste Daroussin2015-07-113-4/+7
| | | | |
* | | | | Remove useless use of gotoBaptiste Daroussin2015-07-111-4/+3
| | | | |