From 9e422a2e278b0a18a2553810d1c8ec20e7c2d758 Mon Sep 17 00:00:00 2001 From: Ian Lance Taylor Date: Mon, 3 Apr 2000 19:19:11 +0000 Subject: [PATCH] 2000-04-03 H.J. Lu * cache.c (bfd_open_file): Unlink the output file only if it has none zero size. --- bfd/ChangeLog | 5 +++++ bfd/cache.c | 23 +++++++++++++++++++---- 2 files changed, 24 insertions(+), 4 deletions(-) diff --git a/bfd/ChangeLog b/bfd/ChangeLog index 85f0981d6fa..552b8889ea6 100644 --- a/bfd/ChangeLog +++ b/bfd/ChangeLog @@ -1,3 +1,8 @@ +2000-04-03 H.J. Lu + + * cache.c (bfd_open_file): Unlink the output file only if + it has none zero size. + 2000-04-01 Ken Block USG * elf64-alpha.c (elf64_alpha_relocate_section): Don't emit diff --git a/bfd/cache.c b/bfd/cache.c index b28de4bc4b9..ad3140e1ed7 100644 --- a/bfd/cache.c +++ b/bfd/cache.c @@ -285,10 +285,25 @@ bfd_open_file (abfd) } else { - /* Create the file. Unlink it first, for the convenience of - operating systems which worry about overwriting running - binaries. */ - unlink (abfd->filename); + /* Create the file. + + Some operating systems won't let us overwrite a running + binary. For them, we want to unlink the file first. + + However, gcc 2.95 will create temporary files using + O_EXCL and tight permissions to prevent other users from + substituting other .o files during the compilation. gcc + will then tell the assembler to use the newly created + file as an output file. If we unlink the file here, we + open a brief window when another user could still + substitute a file. + + So we unlink the output file if and only if it has + non-zero size. */ + struct stat s; + + if (stat (abfd->filename, &s) == 0 && s.st_size != 0) + unlink (abfd->filename); abfd->iostream = (PTR) fopen (abfd->filename, FOPEN_WB); abfd->opened_once = true; } -- 2.30.2