From: Ian Lance Taylor <ian@airs.com>
Date: Mon, 3 Apr 2000 19:19:11 +0000 (+0000)
Subject: 2000-04-03  H.J. Lu  <hjl@gnu.org>
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=9e422a2e278b0a18a2553810d1c8ec20e7c2d758;p=binutils-gdb.git

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 85f0981d6fa..552b8889ea6 100644
--- 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 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;
 	}