summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | MFHGlen Barber2016-01-120-0/+0
|\| | | | | | | Sponsored by: The FreeBSD Foundation
* | MFHGlen Barber2016-01-120-0/+0
|\| | | | | | | Sponsored by: The FreeBSD Foundation
* | MFHGlen Barber2016-01-070-0/+0
|\| | | | | | | Sponsored by: The FreeBSD Foundation
* | MFHGlen Barber2016-01-050-0/+0
|\| | | | | | | Sponsored by: The FreeBSD Foundation
* | MFH r289384-r293170Glen Barber2016-01-045-14/+44
|\| | | | | | | Sponsored by: The FreeBSD Foundation
| * Simplify code for parsing extra groupsBaptiste Daroussin2015-12-291-1/+1
| |
| * Remove useless assignement of linelenBaptiste Daroussin2015-12-291-2/+0
| |
| * Restore dryrun support for pw groupmodBaptiste Daroussin2015-12-281-0/+5
| |
| * pw_checkname since the beginning if too strict on GECOS field,Baptiste Daroussin2015-12-021-1/+2
| | | | | | | | | | | | | | | | | | | | | | relax it a bit so gecos can be used to store multibytes data. This was unseen before FreeBSD 10.2 as this validation function was motly unused since FreeBSD 10.2 the usage of this function has been generalized to improve validation. Reported by: des MFC after: 1 week
| * Fix handling of numeric-only names with pw lockBaptiste Daroussin2015-12-022-9/+32
| | | | | | | | | | | | | | Add a regression test about it PR: 204968 MFC after: 1 week
| * In pw_userlock, set 'name' to NULL when we encounter an all number stringXin LI2015-10-301-2/+3
| | | | | | | | | | | | | | | | because it is also used as an indicator of whether a name or an UID is being used and we may have undefined results as 'name' may contain uninitialized stack contents. MFC after: 2 weeks
| * Fix unlikely memory leak.Bryan Drewery2015-10-291-1/+3
| | | | | | | | | | | | | | It is unlikely since the first check in the function is that dir[0] is '/', but later code changes may make it real. Coverity CID: 1332104
| * Merge from headBaptiste Daroussin2015-10-2822-1668/+2350
| |\ | | | | | | | | | Sponsored by: Gandi.net
| | * Initialize `quiet` to false so `pw groupnext` again prints out the next gidEnji Cooper2015-10-191-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | by default Reported by: Florian Degner <f.degner@gmx.de> MFC after: 1 week PR: 203876 Sponsored by: EMC / Isilon Storage Division
* | | MFH to r289370Glen Barber2015-10-151-2/+0
|\ \ \ | | |/ | |/| | | | Sponsored by: The FreeBSD Foundation
| * | Refactor the test/ Makefiles after recent changes to bsd.test.mk (r289158) andEnji Cooper2015-10-121-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | netbsd-tests.test.mk (r289151) - Eliminate explicit OBJTOP/SRCTOP setting - Convert all ad hoc NetBSD test integration over to netbsd-tests.test.mk - Remove unnecessary TESTSDIR setting - Use SRCTOP where possible for clarity MFC after: 2 weeks Sponsored by: EMC / Isilon Storage Divison
* | | Merge from headBaptiste Daroussin2015-10-100-0/+0
|\| |
* | | Merge from headBaptiste Daroussin2015-10-091-11/+11
|\| |
| * | Fix a repeated typo: rootir -> rootdir.Warren Block2015-10-091-11/+11
| | | | | | | | | | | | | | | Approved by: bapt MFC after: 1 week
* | | Remerge as previous one was brokenBaptiste Daroussin2015-10-030-0/+0
|\| |
* | | Merge from headBaptiste Daroussin2015-10-011-1/+1
|\| |
| * | Regression: fix usershow -7Baptiste Daroussin2015-09-141-1/+1
| | | | | | | | | | | | | | | Submitted by: Dan McGregor (via IRC) MFC after: 2 days
* | | Finish merging from head, messed up in previous attemptBaptiste Daroussin2015-09-1223-1670/+2496
|\| |
* | | Merge from headBaptiste Daroussin2015-09-1217-710/+957
|\| |
| * | Regression: fix pw usermod -dBaptiste Daroussin2015-09-121-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | Mark the user has having been edited if -d option is passed to usermod and so the request change of home directory actually happen PR: 203052 Reported by: lenzi.sergio@gmail.com MFC after: 2 days
| * | Fix err pointer not initialized to NULL resultingBaptiste Daroussin2015-08-212-0/+16
| | | | | | | | | | | | Reported by: "O. Hartmann" <ohartman@zedat.fu-berlin.de>
| * | Fix typo in regression testBaptiste Daroussin2015-08-211-1/+1
| | |
| * | Fix /home symlink creationBaptiste Daroussin2015-08-212-1/+14
| | | | | | | | | | | | Add regression test about it
| * | Fix useradd regression:Baptiste Daroussin2015-08-212-2/+67
| | | | | | | | | | | | | | | | | | | | | | | | 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-142-0/+16
| | | | | | | | | | | | Reported by: gjb
| * | Add regression tests for a bug reported in stable/10Baptiste Daroussin2015-08-051-0/+9
| | | | | | | | | | | | | | | While pw(8) on head is not affected it is worth adding more regression tests ensuring this bug will not happen unnoticed in the futur
| * | 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
| | |
| * | Convert the year used for regression test fro 2043 to 2037Baptiste Daroussin2015-08-021-11/+11
| | | | | | | | | | | | This makes the regression tests pass on systems where time_t is 32bits
| * | 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
| | |
| * | Add regression tests about adding already existsing groups/usersBaptiste Daroussin2015-08-022-0/+22
| | |
| * | 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-0214-1490/+2011
| |/ | | | | | | | | | | | | | | | | | | | | | | | | 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 netbsd tests on pw(8)Baptiste Daroussin2015-08-021-11/+0
| | | | | | | | | | | | First they are redundant with the tests we currently have on pw(8) Second they to modify the host database instead of being self contained withing the test directory
| * 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
| |