summaryrefslogtreecommitdiffstats
path: root/pw/pw_user.c
Commit message (Collapse)AuthorAgeFilesLines
...
* 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-071-4/+4
| | | | Check early that the new name fits MAXLOGNAME and store it in pwconf
* Refactor input validationBaptiste Daroussin2015-06-071-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 pwconfBaptiste Daroussin2015-06-071-4/+3
|
* Handle -7 via gloval pwconfBaptiste Daroussin2015-06-071-6/+5
|
* Handle pretty print (-P) via global pwconfBaptiste Daroussin2015-06-071-12/+6
|
* Handle dryrun (-N) via global pwconfBaptiste Daroussin2015-06-071-5/+3
|
* Add a new global struct pwconf to store etcpath, rootdir and struct userconfBaptiste Daroussin2015-06-071-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 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-071-8/+9
| | | | to char *, change pw_checkname to directly take char * in input
* New pw -R rootdir optionBaptiste Daroussin2015-06-031-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 Daroussin2015-05-311-15/+15
| | | | use %u for uid_t and gid_t
* Cast -1 to uid_t and bump WARNING level to 3Baptiste Daroussin2015-05-311-2/+2
|
* Make pw_user()/pw_group() more consitent about errorsBaptiste Daroussin2015-05-311-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 Daroussin2015-05-091-7/+7
|
* Use snprintf(3) instead of sprintf(3)Baptiste Daroussin2015-05-091-8/+7
| | | | Remove useless "else"
* Remove some uneeded headersBaptiste Daroussin2015-05-091-1/+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
|
* Revert r277652Baptiste Daroussin2015-01-261-4/+1
| | | | | | 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-241-1/+4
| | | | | PR: 196514 MFC after: 1 week
* Ensure pw userdel -u <invalid> do not try to remove rootBaptiste Daroussin2014-10-281-0/+3
| | | | | | Check the uid passed is actually a number as early as possible MFC after: 1 week
* Fix a regression in pw usermod -G listBaptiste Daroussin2014-10-281-1/+19
| | | | | | | | | | 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
* Two more places where login_setcryptfmt() defaults to MD5 were missedDag-Erling Smørgrav2014-10-091-2/+2
| | | | | | in r252688. MFC after: 3 days
* pw: fix up deletion of users from groupsMateusz Guzik2014-06-271-4/+3
| | | | | | | | | Previuosly given 'foo,bar' members, removing 'foo' would result in an infinite loop. PR: 191427 Submitted by: Voradesh Yenbut <yenbut cs.washington.edu> MFC after: 1 week
* Fix pw(8) deletion of group "username" on userdel even if group "username"Devin Teske2014-03-131-1/+9
| | | | | | | | | | | | is not associated with user "username". E.g., user "foo" has primary group "wheel" and is unassociated with group "foo", yet userdel would delete the group "foo" when deleting user "foo" (despite the fact that user "foo" is not associated with group "foo" in any way). Patch committed with minor style(9) changes. PR: bin/169471 Submitted by: Alexander Pyhalov <apyhalov@gmail.com>
* Part 2 of bug 187310.. had to commit separately due to local confusion.Julian Elischer2014-03-061-15/+21
| | | | | | | | Don't let pw crash when give certain input. PR: 187310 Submitted by: Kim Shrier MFC after: 1 week
* Fall back to sha512 if passwd_format is not set.Dag-Erling Smørgrav2013-07-041-1/+1
| | | | MFC after: 3 days
* There is no reason to disallow setting the password or account expiryDag-Erling Smørgrav2013-07-031-4/+0
| | | | | | date to the current date. MFC after: 3 days
* Fix -Wunsequenced warningTim Kientzle2013-06-291-1/+1
| | | | Submitted by: dt71@gmx.com
* Prevent a null pointer dereference in pw userdel when deletingDaniel Eischen2013-02-011-1/+1
| | | | a user whose group != username.
* pw: free group returned by gr_addMateusz Guzik2013-01-061-0/+1
|
* Simplify the code by using the new gr_add functionBaptiste Daroussin2012-12-271-15/+9
|
* Simplify copying of group members by using memcpyBaptiste Daroussin2012-12-261-2/+1
| | | | Submitted by: Christoph Mallon <christoph.mallon@gmx.de>
* Fix off-by-one error in memory allocation: j entries, one new and a nullBaptiste Daroussin2012-12-261-1/+1
| | | | | | terminator is j + 2. Submitted by: Christoph Mallon <christoph.mallon@gmx.de>
* Fix creating a user and adding it to a groupBaptiste Daroussin2012-12-261-7/+8
| | | | Reported by: "Sam Fourman Jr." <sfourman@gmail.com>, dim
* Avoid overflow of file bufferEitan Adler2012-12-051-1/+1
| | | | | | Submitted by: db Approved by: cperciva MFC after: 2 weeks
* Teach pw(8) about how to use pw/gr API to reduce code duplicationBaptiste Daroussin2012-10-301-7/+51
| | | | MFC after: 2 months
* Do not treat empty name as an uid 0Baptiste Daroussin2012-10-011-1/+2
| | | | Reported by: Robert Bonomi <bonomi@mail.r-bonomi.com>
* Remove useless atoi(3), previous strspn(3) makes sure that a_name->val is aBaptiste Daroussin2012-10-011-2/+1
| | | | | | | | | number. This also allow pw user show to work as expected. PR: bin/172112 Submitted by: "Ilya A. Arkhipov" <rum1cro@yandex.ru> MFC after: 1 month
* Handle NULL return from crypt(3). Mostly from DragonFlyKevin Lo2012-02-221-1/+5
|
* In usr.sbin/pw/pw_user.c, use the correct printf length modifier for aDimitry Andric2011-12-171-1/+1
| | | | | | ptrdiff_t. MFC after: 1 week
* Backout r223115 which potentially caused a POLA violation, by restoringXin LI2011-07-311-1/+1
| | | | | | | | historic behavior (create the default base directory in pw.conf) before I came up with a better fix for this. Requested by: nwhitehorn Approved by: re (kib)
* Don't attempt to create the base directory when -d is specified.Xin LI2011-06-151-0/+1272
| | | | MFC after: 1 month
* Create the altix project branch. The altix project will add supportMarcel Moolenaar2010-03-101-1274/+0
| | | | | | | for the SGI Altix 350 to FreeBSD/ia64. The hardware used for porting is a two-module system, consisting of a base compute module and a CPU expansion module. SGI's NUMAFlex architecture can be an excellent platform to test CPU affinity and NUMA-aware features in FreeBSD.
* Make pw(8) build without <utmp.h>.Ed Schouten2009-12-061-5/+0
| | | | | | The size of the username record in utmp files should not influence the maximum username length. Right now ut_user/ut_name is big enough, so in this case it's dead code anyway.