summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* Remove dead codeBaptiste Daroussin2015-07-051-15/+0
|
* Also validate hours via strptime_l(3)Baptiste Daroussin2015-07-051-51/+40
| | | | | Simplify the code, by only using one parser, ensure the dates (hours and dates) are valid
* Validate expiration datesBaptiste Daroussin2015-07-051-0/+288
| | | | Use strptime_l(3) to validate the dates provided in input
* Fix validation of crypted passwordBaptiste Daroussin2015-07-041-7/+6
| | | | Small cleanups
* Also validate inputs of pw groupmod -h and groupmod -HBaptiste Daroussin2015-07-041-46/+46
|
* Validate input of pw usermod -h and pwusermod -HBaptiste Daroussin2015-07-043-60/+95
| | | | | | | Push the code that set the password into a separate function to improve readability Add regression tests about pw usermod -h and pw usermod -H
* When passwd or group information is changed (by pw, vipw, chpass, ...)Renato Botelho2015-07-022-3/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | temporary file is created and then a rename() call move it to official file. This operation didn't have any check to make sure data was written to disk and if a power cycle happens system could end up with a 0 length passwd or group database. There is a pfSense bug with more infor about it: https://redmine.pfsense.org/issues/4523 The following changes were made to protect passwd and group operations: * lib/libutil/gr_util.c: - Replace mkstemp() by mkostemp() with O_SYNC flag to create temp file - After rename(), fsync() call on directory for faster result * lib/libutil/pw_util.c - Replace mkstemp() by mkostemp() with O_SYNC flag to create temp file * usr.sbin/pwd_mkdb/pwd_mkdb.c - Added O_SYNC flag on dbopen() calls - After rename(), fsync() call on directory for faster result * lib/libutil/pw_util.3 - pw_lock() returns a file descriptor to master password file on success Differential Revision: https://reviews.freebsd.org/D2978 Approved by: bapt Sponsored by: Netgate
* Change the documentation to reflect where the -V option should be usedBaptiste Daroussin2015-06-141-2/+2
| | | | Suggested by: kientzle
* Add META_MODE support.Simon J. Gerraty2015-06-130-0/+0
|\ | | | | | | | | | | | | | | | | | | | | Off by default, build behaves normally. WITH_META_MODE we get auto objdir creation, the ability to start build from anywhere in the tree. Still need to add real targets under targets/ to build packages. Differential Revision: D2796 Reviewed by: brooks imp
| * Merge sync of headSimon J. Gerraty2015-05-2710-263/+214
| |\
| * | Merge from headSimon J. Gerraty2015-05-260-0/+0
| |\|
| * | Merge from head@274682Simon J. Gerraty2014-11-196-11/+129
| |\ \
| * \ \ Merge head from 7/28Simon J. Gerraty2014-08-194-21/+9
| |\ \ \
| * \ \ \ Merge from headSimon J. Gerraty2014-05-081-0/+51
| |\ \ \ \
| * \ \ \ \ Merge headSimon J. Gerraty2014-04-285-21/+1031
| |\ \ \ \ \
| * | | | | | Merge headSimon J. Gerraty2014-04-270-0/+0
| |\| | | | |
| * | | | | | Merge from headSimon J. Gerraty2013-09-053-6/+1139
| |\ \ \ \ \ \
| * \ \ \ \ \ \ sync from headSimon J. Gerraty2013-04-122-75/+120
| |\ \ \ \ \ \ \
| * \ \ \ \ \ \ \ Sync with HEAD.David E. O'Brien2013-02-0813-250/+691
| |\ \ \ \ \ \ \ \
| * \ \ \ \ \ \ \ \ Correct the r242545 sync with head@242525.David E. O'Brien2013-02-070-0/+0
| |\ \ \ \ \ \ \ \ \
| * | | | | | | | | | Sync from headSimon J. Gerraty2012-11-0417-21/+2733
| |\| | | | | | | | |
| * | | | | | | | | | MFC: r237279: install filemon.hDavid E. O'Brien2012-06-200-0/+0
| |\ \ \ \ \ \ \ \ \ \
| * \ \ \ \ \ \ \ \ \ \ MFC: r236422, r236592, r236593, r236594, r236620, r236621, r236622, & r236637David E. O'Brien2012-06-060-0/+0
| |\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | filemon(4)
* | | | | | | | | | | | | backout remove of -q option for pw [user|group] nextBaptiste Daroussin2015-06-084-9/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | While the return code is broken, some corner case usage depends on the functionnality, so backout until we get better regression tests covering those corner case usage.
* | | | | | | | | | | | | Fix typoBaptiste Daroussin2015-06-081-1/+1
| | | | | | | | | | | | |
* | | | | | | | | | | | | Fix mistakes than came along with r284139Baptiste Daroussin2015-06-071-2/+2
| | | | | | | | | | | | |
* | | | | | | | | | | | | Remove '-q' support for pw [user|group] nextBaptiste Daroussin2015-06-074-20/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the intent of -q in this command is to return as exit status the value of the next group/user id, which does not make sense given exit status are limited to values between 0 and 255.
* | | | | | | | | | | | | Fix setting uid/gid min/max via pwBaptiste Daroussin2015-06-071-4/+13
| | | | | | | | | | | | |
* | | | | | | | | | | | | Fix generating configuration fileBaptiste Daroussin2015-06-073-5/+6
| | | | | | | | | | | | |
* | | | | | | | | | | | | Fix duplicate checkingBaptiste Daroussin2015-06-074-18/+21
| | | | | | | | | | | | |
* | | | | | | | | | | | | Remove uneeded code (already done by pw_make_v7)Baptiste Daroussin2015-06-071-3/+0
| | | | | | | | | | | | |
* | | | | | | | | | | | | In case of rename validate the length of the new nameBaptiste Daroussin2015-06-074-9/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Check early that the new name fits MAXLOGNAME and store it in pwconf
* | | | | | | | | | | | | Refactor input validationBaptiste Daroussin2015-06-074-102/+108
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Mutualize code to validate inputs of both 'user' and 'group' command Test that the input name fits into MAXLOGNAME
* | | | | | | | | | | | | Finish converting -7 to pwconfBaptiste Daroussin2015-06-071-4/+3
| | | | | | | | | | | | |
* | | | | | | | | | | | | Handle -7 via gloval pwconfBaptiste Daroussin2015-06-073-6/+9
| | | | | | | | | | | | |
* | | | | | | | | | | | | Initialize conf using menset(3)Baptiste Daroussin2015-06-071-3/+1
| | | | | | | | | | | | |
* | | | | | | | | | | | | Handle pretty print (-P) via global pwconfBaptiste Daroussin2015-06-074-21/+18
| | | | | | | | | | | | |
* | | | | | | | | | | | | Handle dryrun (-N) via global pwconfBaptiste Daroussin2015-06-074-7/+11
| | | | | | | | | | | | |
* | | | | | | | | | | | | Sort headersBaptiste Daroussin2015-06-071-2/+2
| | | | | | | | | | | | |
* | | | | | | | | | | | | Handle -C and -Y locally and stop adding them to arglistBaptiste Daroussin2015-06-071-5/+15
| | | | | | | | | | | | |
* | | | | | | | | | | | | Add a new global struct pwconf to store etcpath, rootdir and struct userconfBaptiste Daroussin2015-06-077-113/+86
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Do not add anymore -R and -V to arglist Add an error message if both -V and -R are set in arguments
* | | | | | | | | | | | | Fix pw userdel -r not deleting homedirBaptiste Daroussin2015-06-071-1/+1
| | | | | | | | | | | | |
* | | | | | | | | | | | | Test explicitly the result of strcmp to be consistent with the rest of the codeBaptiste Daroussin2015-06-071-1/+1
| | | | | | | | | | | | |
* | | | | | | | | | | | | Improve readability by reducing indentations levelsBaptiste Daroussin2015-06-071-9/+11
| | | | | | | | | | | | |
* | | | | | | | | | | | | Remove one level of indentationBaptiste Daroussin2015-06-071-7/+6
| | | | | | | | | | | | |
* | | | | | | | | | | | | Move user deletion code into a separate function to improve readabilityBaptiste Daroussin2015-06-071-106/+115
| | | | | | | | | | | | |
* | | | | | | | | | | | | Instead of always casting the pw_checkname input to u_char * and casting it backBaptiste Daroussin2015-06-073-11/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | to char *, change pw_checkname to directly take char * in input
* | | | | | | | | | | | | revert r283969,283970 not needed anymore after r283981Baptiste Daroussin2015-06-042-29/+3
| | | | | | | | | | | | |
* | | | | | | | | | | | | Add a pw_mkdb2(3) function which does the same thing as pw_mkdb(3) exceptBaptiste Daroussin2015-06-032-3/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | it takes a new argument allowing to specify the endianness of the database to generate Differential Revision: https://reviews.freebsd.org/D2730 Reviewed by: ian
* | | | | | | | | | | | | Document pw(8) '-R' option, added in r283961.Glen Barber2015-06-031-1/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | MFC after: 1 week X-MFC-after: r283961 Sponsored by: The FreeBSD Foundation