2000-04-03 H.J. Lu <hjl@gnu.org>

	* cache.c (bfd_open_file): Unlink the output file only if
	it has none zero size.

diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index 85f0981d6faa06dbb3f6e50f14e4fb101ef3bfe7..552b8889ea6f328090ed366e3567e4d406e86842 100644 (file)
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,3 +1,8 @@
+2000-04-03  H.J. Lu  <hjl@gnu.org>
+
+       * cache.c (bfd_open_file): Unlink the output file only if
+       it has none zero size.
+
 2000-04-01  Ken Block USG  <block@zk3.dec.com>
 
        * elf64-alpha.c (elf64_alpha_relocate_section): Don't emit

diff --git a/bfd/cache.c b/bfd/cache.c
index b28de4bc4b96413baff41de25912342713016d4c..ad3140e1ed7305d2559fa06450d736934b2374bb 100644 (file)
--- 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;
        }