From owner-freebsd-current Tue Nov 20 6:12:57 2001 Delivered-To: freebsd-current@freebsd.org Received: from heechee.tobez.org (254.adsl0.ryv.worldonline.dk [213.237.10.254]) by hub.freebsd.org (Postfix) with ESMTP id 44CBC37B405; Tue, 20 Nov 2001 06:12:53 -0800 (PST) Received: by heechee.tobez.org (Postfix, from userid 1001) id A8498542E; Tue, 20 Nov 2001 15:12:50 +0100 (CET) Date: Tue, 20 Nov 2001 15:12:50 +0100 From: Anton Berezin To: Ruslan Ermilov Cc: Max Khon , current@FreeBSD.org Subject: Re: misc/15421 (was: Re: initgroups) Message-ID: <20011120151250.C36033@heechee.tobez.org> Mail-Followup-To: Anton Berezin , Ruslan Ermilov , Max Khon , current@FreeBSD.org References: <20011114021956.B10325@iclub.nsu.ru> <3BF19EA9.3FC5F040@mindspring.com> <20011119181949.R32927@sunbay.com> <20011119222854.B38492@iclub.nsu.ru> <20011120150239.D7645@sunbay.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5i In-Reply-To: <20011120150239.D7645@sunbay.com>; from ru@FreeBSD.org on Tue, Nov 20, 2001 at 03:02:39PM +0200 Sender: owner-freebsd-current@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On Tue, Nov 20, 2001 at 03:02:39PM +0200, Ruslan Ermilov wrote: > On Mon, Nov 19, 2001 at 10:28:55PM +0600, Max Khon wrote: > > I asked tobez (he is an originator and he took responsibility on > > this PR) and he said that src/ must be audited also -- he said that > > some initgroups() callers do not print error message because > > initgroups() did this previously. > > > > I'll try to do this before this weekend and I will post combined > > patch to audit@ > While this is indeed a good thing to do, this is completely unrelated to > the above mentioned problem, and should be done separately. Here's the > list of src/ files that do not check the return value of initgroups(3), > and may need to be fixed, but some of them explicitly ignore the result > to indicate the fact they consider this error non-fatal. > libexec/ftpd/ftpd.c > libexec/rexecd/rexecd.c > usr.bin/calendar/calendar.c > usr.sbin/inetd/inetd.c There used to be *many* more problematic files. Please see http://www.freebsd.org/cgi/getmsg.cgi?fetch=801566+0+/usr/local/www/db/text/2001/freebsd-stable/20010722.freebsd-stable To my knowledge, only printjob.c was fixed, though I have not looked into every file in the list since then. But as I said in the private message, I do not feel strongly about this, and I think that the fix can be safely committed. I do not think these things are quite unrelated, though. :-) Cheers, \Anton. -- | Anton Berezin | FreeBSD: The power to serve | | catpipe Systems ApS _ _ |_ | http://www.FreeBSD.org | | tobez@catpipe.net (_(_|| | tobez@FreeBSD.org | | +45 7021 0050 | Private: tobez@tobez.org | To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-current" in the body of the message