From owner-cvs-src@FreeBSD.ORG  Tue May 17 04:16:31 2005
Return-Path: <owner-cvs-src@FreeBSD.ORG>
Delivered-To: cvs-src@freebsd.org
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id 270ED16A4CE
	for <cvs-src@FreeBSD.org>; Tue, 17 May 2005 04:16:31 +0000 (GMT)
Received: from mailbox4.ucsd.edu (mailbox4.ucsd.edu [132.239.1.56])
	by mx1.FreeBSD.org (Postfix) with ESMTP id B8AC843D95
	for <cvs-src@FreeBSD.org>; Tue, 17 May 2005 04:16:30 +0000 (GMT)
	(envelope-from okumoto@ucsd.edu)
Received: from smtp.ucsd.edu (smtp-a.ucsd.edu [132.239.1.49])
	by mailbox4.ucsd.edu (8.13.3/8.13.3) with ESMTP id j4H4GCg6031367
	(version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO);
	Mon, 16 May 2005 21:16:17 -0700 (PDT)
Received: from ucsd.edu (adsl-63-199-247-178.dsl.sndg02.pacbell.net
	[63.199.247.178])
	by smtp.ucsd.edu (8.12.10/8.9.3) with ESMTP id j4H4G3Tg006653;
	Mon, 16 May 2005 21:16:08 -0700 (PDT)
Message-ID: <42897003.2090005@ucsd.edu>
Date: Mon, 16 May 2005 21:16:03 -0700
From: Max Okumoto <okumoto@ucsd.edu>
User-Agent: Mozilla/5.0 (X11; U; FreeBSD i386; en-US; rv:1.4) Gecko/20030824
X-Accept-Language: en-us, en
MIME-Version: 1.0
To: Max Okumoto <okumoto@ucsd.edu>
References: <200505121545.j4CFjENu078768@repoman.freebsd.org>
	<20050512180743.6z1h22fldwksgw4w@netchild.homeip.net>
	<hfbr7ge2gu.fsf@multivac.sdsc.edu>
In-Reply-To: <hfbr7ge2gu.fsf@multivac.sdsc.edu>
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit
X-Greylisting: NO DELAY (Trusted relay host);
	processed by UCSD_GL-v1.2 on mailbox4.ucsd.edu;
	Mon, 16 May 2005 21:16:17 -0700 (PDT)
X-MailScanner: PASSED (v1.2.8 19588 j4H4GCg6031367 mailbox4.ucsd.edu)
cc: cvs-src@FreeBSD.org
cc: Alexander Leidinger <Alexander@Leidinger.net>
Subject: Re: cvs commit: src/usr.bin/make job.c
X-BeenThere: cvs-src@freebsd.org
X-Mailman-Version: 2.1.1
Precedence: list
List-Id: CVS commit messages for the src tree <cvs-src.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-src>
List-Post: <mailto:cvs-src@freebsd.org>
List-Help: <mailto:cvs-src-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src>,
	<mailto:cvs-src-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Tue, 17 May 2005 04:16:31 -0000

Max Okumoto wrote:
> Alexander Leidinger <Alexander@Leidinger.net> writes:
> 
> [stuff deleted]
>  
>>For those which didn't looked at the src: he can't use mkstemp() since he
>>wants to create a fifo.
>>
>>But "making a copy" and creating mkfifo() out of it looks like overkill to
>>me. Why not using mkdtemp() and creating a fixed name in this temporary
>>directory instead?
>>
>>Bye,
>>Alexander.
> 
> 
> [stuff deleted]
> 
> Hmmm... Didn't think of that :-)  That is better way since I wouldn't
> have to maintain a my own copy of the code.  I will work up a patch.
> 
>                       Max


If I use mkdtemp(), there is a chance that someone with the
same UID could race to build the fifo. Do we care about
races with ourselves?  Or am I just being toooo paranoid? :-)


				Max