summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* libutil: utilize strsep instead of strcat in a loop in gr_makeMateusz Guzik2013-01-131-6/+10
| | | | Submitted by: Christoph Mallon <christoph.mallon gmx.de>
* libutil: move group_line_format into the scop of its only user.Mateusz Guzik2013-01-131-2/+1
| | | | Submitted by: Christoph Mallon <christoph.mallon gmx.de>
* libutil: eliminate 'found' variable in gr_equalMateusz Guzik2013-01-131-8/+5
| | | | Submitted by: Christoph Mallon <christoph.mallon gmx.de>
* pw: free group returned by gr_addMateusz Guzik2013-01-061-0/+1
|
* Simplify pointing dst after the end of all the gr_mem pointers in newgrBaptiste Daroussin2012-12-281-2/+1
| | | | | Submitted by: pjd Reviewed by: db
* errno = ENOMEM was supposed to be removed not return (NULL);Baptiste Daroussin2012-12-281-1/+1
| | | | Submitted by: gcooper
* malloc() sets errno to ENOMEM already.Baptiste Daroussin2012-12-281-5/+1
| | | | Submitted by: Christoph Mallon <christoph.mallon@gmx.de>
* 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.