This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [RFC 4/6] Use mkostemp, not mkstemp

From: Simon Marchi <simark at simark dot ca>
To: Tom Tromey <tom at tromey dot com>, gdb-patches at sourceware dot org
Date: Fri, 28 Sep 2018 23:09:06 -0400
Subject: Re: [RFC 4/6] Use mkostemp, not mkstemp
References: <20180926111130.18956-1-tom@tromey.com> <20180926111130.18956-5-tom@tromey.com>

On 2018-09-26 7:11 a.m., Tom Tromey wrote:
> I noticed that gdb could leak file descriptors coming from mkstemp.
> This patch fixes the problem by importing the gnulib mkostemp instead,
> and then changing gdb to pass O_CLOEXEC.

While this looks like the correct thing to do, I am curious to know if
you encountered an actual issue or this is theoretical.  I don't see
how a fork/exec could happen while one of these temp files are open.

Simon

Follow-Ups:
- Re: [RFC 4/6] Use mkostemp, not mkstemp
  - From: Tom Tromey

References:
- [RFC 0/6] A different approach to startup-with-shell on macOS
  - From: Tom Tromey
- [RFC 4/6] Use mkostemp, not mkstemp
  - From: Tom Tromey

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]