summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* commit correct tested fix for gr_util.cDiane Bruce2013-03-091-46/+58
| | | | Approved by: theraven
* Cleanup gr_add() so it does not leak memDiane Bruce2013-03-072-53/+86
| | | | | | | | | This is part of ongoing work on sbin/pw M libutil.h M gr_util.c Approved by: theraven
* Prevent a null pointer dereference in pw userdel when deletingDaniel Eischen2013-02-011-1/+1
| | | | a user whose group != username.
* IFC @ r245509Neel Natu2013-01-172-19/+17
|\
| * Remove unused variablesEitan Adler2013-01-161-2/+0
| | | | | | | | | | Approved by: cperciva MFC After: 3 days
| * libutil: fix typo in comment for gr_fini.Mateusz Guzik2013-01-131-1/+1
| | | | | | | | Submitted by: Christoph Mallon <christoph.mallon gmx.de>
| * 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>
* | IFC @ r245178Neel Natu2013-01-091-0/+1
|\|
| * pw: free group returned by gr_addMateusz Guzik2013-01-061-0/+1
| |
* | IFC @ r244983.Neel Natu2013-01-0410-230/+362
|\|
| * 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
* | IFC @r243836Neel Natu2012-12-042-0/+5
|\|
| * 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
| |
* | IFC @ r243164Neel Natu2012-11-171-0/+305
|\|
| * 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
* | IFC @ r242684Neel Natu2012-11-1115-15/+2510
|\|
| * 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