summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Do not leave parts of the new group uninitialized in gr_dup().Baptiste Daroussin2012-12-281-0/+4
| | | | | Submitted by: Christoph Mallon <christoph.mallon@gmx.de> Reported by: pjd
* avoid arithmetic on uintptr_tBaptiste Daroussin2012-12-271-1/+1
| | | | | Submitted by: pjd Reviewed by: jilles
* cast to uintptr_t to properly calculate offsetBaptiste Daroussin2012-12-271-1/+1
| | | | | Reported by: mdf Submitted by: db
* Add O_CLOEXEC to flopenBaptiste Daroussin2012-12-272-2/+2
| | | | Requested by: jilles
* Simplify vnextgrent and vnextpwent reusing pw_scan and gr_scan from libutil.Baptiste Daroussin2012-12-271-169/+67
|
* gr_dup: simplify duplication of groupBaptiste Daroussin2012-12-271-21/+20
| | | | Submitted by: db
* - Clean up previous gr_add use malloc instead of callocBaptiste Daroussin2012-12-272-5/+4
| | | | | | - Fix tinderbox error Submitted by: db
* Fix a regression in "pw group show" introduced r242349:Baptiste Daroussin2012-12-271-1/+1
| | | | | | | print a newline after printing each group line. PR: bin/174731 Submitted by: Jan Beich <jbeich@tormail.org>
* Simplify the code by using the new gr_add functionBaptiste Daroussin2012-12-271-15/+9
|
* New gr_add function to provide a clean and safe method to append a new memberBaptiste Daroussin2012-12-272-0/+42
| | | | | | into an existing group. Submitted by: db
* Use flopen(3) instead of open(2) + flock(2)Baptiste Daroussin2012-12-272-9/+4
|
* 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>
* In case of the deletion of a user those whole database has to be regenerated,Baptiste Daroussin2012-12-261-1/+5
| | | | | otherwise the user planned to be deleted remain in the pwd.db while removed from the plain text password file.
* Fix creating a user and adding it to a groupBaptiste Daroussin2012-12-261-7/+8
| | | | Reported by: "Sam Fourman Jr." <sfourman@gmail.com>, dim
* Simplify string duplication: use strdup instead of malloc + strcpyEitan Adler2012-12-051-6/+5
| | | | | | Submitted by: db Approved by: cperciva MFC after: 2 weeks
* Avoid overflowing the file bufferEitan Adler2012-12-051-0/+75
| | | | | | Submitted by: db Approved by: cperciva MFC after: 2 weeks
* Use strdup instead of malloc + strcpyEitan Adler2012-12-051-6/+4
| | | | | | Submitted by: db Approved by: cperciva MFC after: 2 weeks
* Avoid overflow of file bufferEitan Adler2012-12-051-1/+1
| | | | | | Submitted by: db Approved by: cperciva MFC after: 2 weeks
* Remove useless check for NULL prior to free.Eitan Adler2012-12-052-2/+132
| | | | | Approved by: cperciva MFC after: 2 weeks
* In NIS mode first chmod(2) the temporary file and is succeed then rename(2)Baptiste Daroussin2012-11-201-2/+2
|
* only rename(2) after chmod(2) has succeedBaptiste Daroussin2012-11-201-7/+3
| | | | | | report error if chmod(2) fails Reported by: jh
* Correctly set the password file mode after renaming in NIS modeBaptiste Daroussin2012-11-201-0/+2
|
* change mode the group file to 0644 after a successfull rename(2)Baptiste Daroussin2012-11-201-1/+8
|
* Avoid possible null deref if ypclnt_new returns nullEitan Adler2012-11-151-0/+305
| | | | | | | PR: bin/172979 Submitted by: Erik Cederstrand <erik@cederstrand.dk> Approved by: cperciva MFC after: 3 days
* Teach pw(8) about how to use pw/gr API to reduce code duplicationBaptiste Daroussin2012-10-3013-88/+1888
| | | | MFC after: 2 months
* backout r242319, racy and not done in the right placeBaptiste Daroussin2012-10-292-18/+0
| | | | Reported by: Garrett Cooper <yanegomi@gmail.com>
* make pw_init and gr_init fail if the specified master password or group file isBaptiste Daroussin2012-10-292-0/+20
| | | | | | a directory. MFC after: 1 month
* 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
* Make sure that each va_start has one and only one matching va_end,Kevin Lo2012-09-281-0/+214
| | | | especially in error cases.
* Remove a reference to CVS and to freefall from a user-facing man page.Isabell Long2012-09-151-0/+479
| | | | | | | PR: docs/171658 Reported by: Chris Petrik (c dot petrik dot sosa at gmail dot com) Approved by: gabor (mentor) MFC after: 5 days
* Revert user comparison back to user names as some user can share uids (root/toorBaptiste Daroussin2012-06-191-6/+13
| | | | | | | | | | | for example) get the username information from old_pw structures to still allow renaming of a user. Reported by: Claude Buisson <clbuisson@orange.fr> Approved by: des (mentor) MFC after: 3 weeks
* mdoc: add missing width argument to Bl -tag.Joel Dahl2012-06-031-0/+210
|
* A new jail(8) with a configuration file, ultimately to replace the workJamie Gritton2012-04-260-0/+0
|\ | | | | | | | | | | currently done by /etc/rc.d/jail. MFC after: 3 months
| * A new jail(8) with a configuration file, to replace the work currently doneJamie Gritton2010-10-1948-13002/+0
| | | | | | | | by /etc/rc.d/jail.
* | Remove trailing whitespace per mdoc lint warningEitan Adler2012-03-291-0/+221
| | | | | | | | | | | | | | Disussed with: gavin No objection from: doc Approved by: joel MFC after: 3 days
* | When using uidstart in /etc/adduser.conf, get the nextDaniel Eischen2012-02-251-0/+1
| | | | | | | | | | | | | | | | available user id and show it in the "Uid [xxx]" prompt. PR: 163863 Submitted by: Moritz Wilhelmy (mw at wzff dot de) MFC after: 2 weeks
* | Handle NULL return from crypt(3). Mostly from DragonFlyKevin Lo2012-02-221-1/+5
| |
* | readlink len-1Kevin Lo2012-02-141-1/+1
| |
* | Remove unnecessary castKevin Lo2012-02-141-0/+182
| |
* | Detect file modification properly by using tv_nsec.Ed Schouten2012-02-101-1/+2
| | | | | | | | | | | | | | POSIX 2008 standardizes st_mtim, meaning we can simply use nanosecond precision to detect file modification. MFC after: 2 weeks
* | Consensus between bde and pjd seemed to be that if the function namesGuy Helmer2012-01-261-8/+8
| | | | | | | | | | | | are lined up, then any * after a long type should appear after the type instead of being in front of the function name on the following line.
* | Make the comments consistent (capitalization, punctuation, andGuy Helmer2012-01-261-6/+10
| | | | | | | | | | | | format). Requested by bde
* | Restore the parenthesis that are necessary around the constant values.Guy Helmer2012-01-261-7/+7
| | | | | | | | Requested by bde.
* | Fix more disorder in prototypes and constants.Guy Helmer2012-01-161-46/+43
| | | | | | | | | | | | Fix header comments for each section of constants. Fix whitespace in #define lines. Fix unnecessary parenthesis in constants.
* | Move struct pidfh definition into pidfile.c, and leave a forward declarationGuy Helmer2012-01-121-12/+6
| | | | | | | | | | | | | | | | | | for pidfh in libutil.h in its place. This allows us to hide the contents of the pidfh structure, and also allowed removal of the "#ifdef _SYS_PARAM_H" guard from around the pidfile_* function prototypes. Suggested by pjd.
* | More prototype formatting fixes, struct member formatting fixes,Guy Helmer2012-01-121-10/+17
| | | | | | | | | | | | and namespace fix for property_find() prototype. Provided by bde.
* | Fix prototype formatting (indentation, long lines, and continued lines).Guy Helmer2012-01-111-23/+30
| | | | | | | | Requested by bde.
* | Fix namespace issues with prototype parameter names.Guy Helmer2012-01-111-32/+32
| | | | | | | | | | | | Add missing prototype parameter names. Requested by bde.