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 21867d21979aa67a0b010d8a869e634272a7189d..4c146581ebab781796d0d485144e325c5e276e54 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 e476194f0499a9bdd396d98bc116cb5f85ffacb5..8c2f8f34cf3124baa9c1eb8c3ab076338b083de8 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;
 }