From owner-cvs-src@FreeBSD.ORG Sat Aug 27 04:56:39 2005 Return-Path: X-Original-To: cvs-src@FreeBSD.org Delivered-To: cvs-src@FreeBSD.org Received: from green.homeunix.org (freefall.freebsd.org [216.136.204.21]) by hub.freebsd.org (Postfix) with ESMTP id A1E5B16A41F; Sat, 27 Aug 2005 04:56:36 +0000 (GMT) (envelope-from green@green.homeunix.org) Received: from green.homeunix.org (green@localhost [127.0.0.1]) by green.homeunix.org (8.13.3/8.13.1) with ESMTP id j7R4xDmw053098; Sat, 27 Aug 2005 00:59:16 -0400 (EDT) (envelope-from green@green.homeunix.org) Received: (from green@localhost) by green.homeunix.org (8.13.3/8.13.1/Submit) id j7R4xBU6053097; Sat, 27 Aug 2005 00:59:11 -0400 (EDT) (envelope-from green) Date: Sat, 27 Aug 2005 00:59:11 -0400 From: Brian Fundakowski Feldman To: Garance A Drosehn Message-ID: <20050827045911.GB46755@green.homeunix.org> References: <200508241938.j7OJcSNW066686@repoman.freebsd.org> <200508241602.59424.jhb@FreeBSD.org> <20050824205627.GD755@garage.freebsd.pl> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.6i Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, Pawel Jakub Dawidek , cvs-all@FreeBSD.org, John Baldwin Subject: Re: cvs commit: src/usr.bin/pkill pkill.1 pkill.c X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 27 Aug 2005 04:56:39 -0000 On Wed, Aug 24, 2005 at 06:14:15PM -0400, Garance A Drosehn wrote: > At 10:56 PM +0200 8/24/05, Pawel Jakub Dawidek wrote: > >On Wed, Aug 24, 2005, John Baldwin wrote: > >+> On Wednesday 24 August 2005, Pawel Jakub Dawidek wrote: > >+> > Log: > >+> > Modify '-F' option to work nicely with pidfile(3) - a pidfile > >+> > given as an argument has to be locked. > >+> > > >+> > >+> I guess the only thing that makes me uncomfortable about this > >+> change is that it means pkill won't work with 3rd party program's > >+> pid files since they are probably not using pidfile(3). Maybe > >+> have some way of turning off the flock requirement? > > I must admit that I did not quite understand what you had meant in > your earlier messages about changes to `pkill' to match your new > pid-handling library routines. I agree with John, this change > doesn't seem quite right to me. What keeps an arbitrary user from locking the pid file and making a mess of this protocol? It seems less error prone to just make pkill verify the (pid, procname, uid, gid, pgrp) are all in line with what the pidfile implies... to me at least. -- Brian Fundakowski Feldman \'[ FreeBSD ]''''''''''\ <> green@FreeBSD.org \ The Power to Serve! \ Opinions expressed are my own. \,,,,,,,,,,,,,,,,,,,,,,\