2008-08-20 Craig Silverstein <csilvers@google.com>

	* fileread.cc (File_read::open): Do not lock the file unless it
	was successfully opened.

diff --git a/gold/ChangeLog b/gold/ChangeLog
index 21867d2..4c14658 100644 (file)
--- a/gold/ChangeLog
+++ b/gold/ChangeLog
@@ -1,3 +1,8 @@
+2008-08-20  Craig Silverstein  <csilvers@google.com>
+
+       * fileread.cc (File_read::open): Do not lock the file unless it
+       was successfully opened.
+
 2008-08-14  Cary Coutant  <ccoutant@google.com>
 
        * x86_64.cc (Target_x86_64::Relocate::relocat_tls):

diff --git a/gold/fileread.cc b/gold/fileread.cc
index e476194..8c2f8f3 100644 (file)
--- a/gold/fileread.cc
+++ b/gold/fileread.cc
@@ -121,9 +121,9 @@ File_read::open(const Task* task, const std::string& name)
       this->size_ = s.st_size;
       gold_debug(DEBUG_FILES, "Attempt to open %s succeeded",
                  this->name_.c_str());
-    }
 
-  this->token_.add_writer(task);
+      this->token_.add_writer(task);
+    }
 
   return this->descriptor_ >= 0;
 }