Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Split some extra long lines | Baptiste Daroussin | 2015-08-02 | 1 | -5/+10 |
| | |||||
* | Cleanup a bit includes | Baptiste Daroussin | 2015-08-02 | 1 | -10/+15 |
| | |||||
* | Fix regression: report again if a username already exists when creating it | Baptiste Daroussin | 2015-08-02 | 1 | -0/+3 |
| | |||||
* | Rewrite parsing subcommands arguments of pw(8) | Baptiste Daroussin | 2015-08-02 | 1 | -923/+1248 |
| | | | | | | | | | | | | | 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. | ||||
* | Partial revert of r286152 | Baptiste Daroussin | 2015-08-01 | 1 | -4/+4 |
| | | | | More work needed on the cli validation | ||||
* | Validate expiration days and password days from commmand line and pw.conf | Baptiste Daroussin | 2015-08-01 | 1 | -4/+4 |
| | |||||
* | Cast uid/git to uintmax_t when using printf-like functions so the size of | Baptiste Daroussin | 2015-08-01 | 1 | -16/+18 |
| | | | | uid/gid size remains a implementation detail | ||||
* | Reject usermod and userdel if the user concerned is not on the user database | Baptiste Daroussin | 2015-07-28 | 1 | -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 | ||||
* | Fix wrong warning printed after changing or updating NIS users | Baptiste Daroussin | 2015-07-28 | 1 | -2/+2 |
| | | | | | PR: 37672 Submitted by: chris+freebsd@chrullrich.de | ||||
* | Regression fix: allow to create users with uid0 | Baptiste Daroussin | 2015-07-13 | 1 | -1/+1 |
| | | | | Reported by: Jan Mikkelsen <janm@transactionware.com> | ||||
* | Ensure skeldir is abolute path (relatively to the rootdir) | Baptiste Daroussin | 2015-07-12 | 1 | -4/+10 |
| | |||||
* | pw -R <rootdir> userdel can now cleanup installation | Baptiste Daroussin | 2015-07-12 | 1 | -56/+50 |
| | | | | | | | 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 *at | Baptiste Daroussin | 2015-07-12 | 1 | -9/+8 |
| | | | | | | | | | 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 NULL | Baptiste Daroussin | 2015-07-12 | 1 | -4/+1 |
| | |||||
* | Fix regression: ensure when try to create the group and the user with the same | Baptiste Daroussin | 2015-07-11 | 1 | -12/+4 |
| | | | | id if possible and nothing in particular was specified | ||||
* | Replace custom string array with stringlist(3) | Baptiste Daroussin | 2015-07-11 | 1 | -10/+6 |
| | |||||
* | Remove unused argument from pm_passwd | Baptiste Daroussin | 2015-07-11 | 1 | -4/+4 |
| | |||||
* | check the gecos format early: at the moment the -c option is parsed | Baptiste Daroussin | 2015-07-11 | 1 | -4/+3 |
| | |||||
* | Remove useless use of goto | Baptiste Daroussin | 2015-07-11 | 1 | -4/+3 |
| | |||||
* | Isolate pw lock/unlock into a separate function | Baptiste Daroussin | 2015-07-11 | 1 | -44/+78 |
| | |||||
* | homedir can only be populate during useradd | Baptiste Daroussin | 2015-07-11 | 1 | -3/+3 |
| | |||||
* | Make a separate groupdel/userdel from the main function | Baptiste Daroussin | 2015-07-11 | 1 | -17/+31 |
| | |||||
* | Make separate functions to show users and groups | Baptiste Daroussin | 2015-07-11 | 1 | -17/+35 |
| | |||||
* | Move the quiet flag into the configuration structure | Baptiste Daroussin | 2015-07-11 | 1 | -1/+1 |
| | |||||
* | Separate usernext/groupnext from the main functions | Baptiste Daroussin | 2015-07-11 | 1 | -17/+17 |
| | |||||
* | Fix validation of crypted password | Baptiste Daroussin | 2015-07-04 | 1 | -7/+6 |
| | | | | Small cleanups | ||||
* | Validate input of pw usermod -h and pwusermod -H | Baptiste Daroussin | 2015-07-04 | 1 | -60/+63 |
| | | | | | | | 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 | ||||
* | backout remove of -q option for pw [user|group] next | Baptiste Daroussin | 2015-06-08 | 1 | -3/+7 |
| | | | | | | 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. | ||||
* | Remove '-q' support for pw [user|group] next | Baptiste Daroussin | 2015-06-07 | 1 | -7/+3 |
| | | | | | | 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 generating configuration file | Baptiste Daroussin | 2015-06-07 | 1 | -4/+3 |
| | |||||
* | Fix duplicate checking | Baptiste Daroussin | 2015-06-07 | 1 | -10/+9 |
| | |||||
* | Remove uneeded code (already done by pw_make_v7) | Baptiste Daroussin | 2015-06-07 | 1 | -3/+0 |
| | |||||
* | In case of rename validate the length of the new name | Baptiste Daroussin | 2015-06-07 | 1 | -4/+4 |
| | | | | Check early that the new name fits MAXLOGNAME and store it in pwconf | ||||
* | Refactor input validation | Baptiste Daroussin | 2015-06-07 | 1 | -58/+37 |
| | | | | | Mutualize code to validate inputs of both 'user' and 'group' command Test that the input name fits into MAXLOGNAME | ||||
* | Finish converting -7 to pwconf | Baptiste Daroussin | 2015-06-07 | 1 | -4/+3 |
| | |||||
* | Handle -7 via gloval pwconf | Baptiste Daroussin | 2015-06-07 | 1 | -6/+5 |
| | |||||
* | Handle pretty print (-P) via global pwconf | Baptiste Daroussin | 2015-06-07 | 1 | -12/+6 |
| | |||||
* | Handle dryrun (-N) via global pwconf | Baptiste Daroussin | 2015-06-07 | 1 | -5/+3 |
| | |||||
* | Add a new global struct pwconf to store etcpath, rootdir and struct userconf | Baptiste Daroussin | 2015-06-07 | 1 | -14/+16 |
| | | | | | 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 homedir | Baptiste Daroussin | 2015-06-07 | 1 | -1/+1 |
| | |||||
* | Test explicitly the result of strcmp to be consistent with the rest of the code | Baptiste Daroussin | 2015-06-07 | 1 | -1/+1 |
| | |||||
* | Improve readability by reducing indentations levels | Baptiste Daroussin | 2015-06-07 | 1 | -9/+11 |
| | |||||
* | Remove one level of indentation | Baptiste Daroussin | 2015-06-07 | 1 | -7/+6 |
| | |||||
* | Move user deletion code into a separate function to improve readability | Baptiste Daroussin | 2015-06-07 | 1 | -106/+115 |
| | |||||
* | Instead of always casting the pw_checkname input to u_char * and casting it back | Baptiste Daroussin | 2015-06-07 | 1 | -8/+9 |
| | | | | to char *, change pw_checkname to directly take char * in input | ||||
* | New pw -R rootdir option | Baptiste Daroussin | 2015-06-03 | 1 | -11/+33 |
| | | | | | | | | This allows to set an alternate root directory in which the users/groups will be manipulated Requested by: gjb, ian Tested by: gjb | ||||
* | Remove useless cast in printf and printf-like functions: | Baptiste Daroussin | 2015-05-31 | 1 | -15/+15 |
| | | | | use %u for uid_t and gid_t | ||||
* | Cast -1 to uid_t and bump WARNING level to 3 | Baptiste Daroussin | 2015-05-31 | 1 | -2/+2 |
| | |||||
* | Make pw_user()/pw_group() more consitent about errors | Baptiste Daroussin | 2015-05-31 | 1 | -25/+15 |
| | | | | | | | | 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) | ||||
* | Replace sprintf(3) with snprintf(3) | Baptiste Daroussin | 2015-05-09 | 1 | -7/+7 |
| |