From: John Darrington <john@darrington.wattle.id.au>
Date: Thu, 12 Apr 2018 14:07:02 +0000 (+0100)
Subject: Stop the assembler from overwriting its output file.
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=67f846b59b32f3d704c601669409c2584383fea9;p=binutils-gdb.git

Stop the assembler from overwriting its output file.

	* as.c (main): Fail if the output is the same as one of the input
	files.
	* testsuite/gas/all/gas.exp: Test the new feature.
---

diff --git a/gas/ChangeLog b/gas/ChangeLog
index f1c081aaf87..d87d3e793a9 100644
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,3 +1,9 @@
+2018-04-12  John Darrington  <john@darrington.wattle.id.au>
+
+	* as.c (main): Fail if the output is the same as one of the input
+	files.
+	* testsuite/gas/all/gas.exp: Test the new feature.
+
 2018-04-12  Nick Clifton  <nickc@redhat.com>
 
 	* po/es.po: Updated Spanish translation.
diff --git a/gas/as.c b/gas/as.c
index f6da1b1a1d4..6e8ec567855 100644
--- a/gas/as.c
+++ b/gas/as.c
@@ -1171,6 +1171,7 @@ int
 main (int argc, char ** argv)
 {
   char ** argv_orig = argv;
+  struct stat sob;
 
   int macro_strip_at;
 
@@ -1218,6 +1219,27 @@ main (int argc, char ** argv)
   /* Call parse_args before any of the init/begin functions
      so that switches like --hash-size can be honored.  */
   parse_args (&argc, &argv);
+
+  if (argc > 1 && stat (out_file_name, &sob) == 0)
+    {
+      int i;
+
+      for (i = 1; i < argc; ++i)
+	{
+	  struct stat sib;
+
+	  if (stat (argv[i], &sib) == 0)
+	    {
+	      if (sib.st_ino == sob.st_ino)
+		{
+		  /* Don't let as_fatal remove the output file!  */
+		  out_file_name = NULL;
+		  as_fatal (_("The input and output files must be distinct"));
+		}
+	    }
+	}
+    }
+
   symbol_begin ();
   frag_init ();
   subsegs_begin ();
diff --git a/gas/testsuite/gas/all/gas.exp b/gas/testsuite/gas/all/gas.exp
index 0355a03fa44..b3d662a9f23 100644
--- a/gas/testsuite/gas/all/gas.exp
+++ b/gas/testsuite/gas/all/gas.exp
@@ -73,6 +73,8 @@ if {    ![istarget alpha*-*-*vms*]
 gas_test_error "equiv1.s" "" ".equiv for symbol already set to another one"
 gas_test_error "equiv2.s" "" ".equiv for symbol already set to an expression"
 
+gas_test_error "none.s" "-o $srcdir/$subdir/none.s" "Output file must be distinct from input"
+
 # .equ works differently on some targets.
 case $target_triplet in {
     { hppa*-*-* } { }