[haiku] Re: File system races?

  • From: David Given <dg@xxxxxxxxxxx>
  • To: haiku@xxxxxxxxxxxxx
  • Date: Mon, 07 Aug 2017 08:56:34 +0000

Sure --- where's the best place? I've kind of lost track of which repos do
what.

(On that subject, where do I file bugs on HaikuDepot package issues?)

On Sun, 6 Aug 2017, 21:56 Axel Dörfler, <axeld@xxxxxxxxxxxxxxxx> wrote:

Am 06/08/2017 um 13:35 schrieb David Given:
So I reckon there are two bugs here:

(a) I shouldn't be using mktemp(); I should be using mkstemp() instead
which atomically creates the file, guaranteeing uniqueness --- although
in fairness, most of this code was written in the 1980s, when mkstemp()
didn't exist;

Yep, switching to mkstemp() definitely makes sense. Massively parallel
builds weren't en vogue back then either :-)

(b) Haiku really shouldn't be doing that.

Yes, it probably shouldn't. Current BSD implementations have replaced
rand() with arc4random_uniform(); while mkstemp() doesn't suffer from
the race condition, it still uses the same random name generator.

Could you please file a bug report about this?

Bye,
    Axel.

--
┌─── http://cowlark.com ───
│ "There is nothing in the world so dangerous --- and I mean *nothing*
│ --- as a children's story that happens to be true." --- Master Li Kao,
│ _The Bridge of Birds_

Other related posts: