summaryrefslogtreecommitdiff
path: root/gold
diff options
context:
space:
mode:
authorIan Lance Taylor <ian@airs.com>2013-02-11 16:12:59 +0000
committerIan Lance Taylor <ian@airs.com>2013-02-11 16:12:59 +0000
commit84ad5f0d91872bef004b3c174365150841c71822 (patch)
tree60b9e279aaf592f0d274e4f357a9f7fc909db2b6 /gold
parent0d33a672223e4c1a2838a4b1c8c31cc199022aed (diff)
downloadbinutils-redhat-84ad5f0d91872bef004b3c174365150841c71822.tar.gz
* descriptors.cc (Descriptors::close_all): New function.
* descriptors.h (class Descriptors): Declare close_all. (close_all_descriptors): New inline function. * plugin.cc: Include "descriptors.h". (Plugin_manager::cleanup): Call close_all_descriptors.
Diffstat (limited to 'gold')
-rw-r--r--gold/ChangeLog8
-rw-r--r--gold/descriptors.cc24
-rw-r--r--gold/descriptors.h10
-rw-r--r--gold/plugin.cc9
4 files changed, 49 insertions, 2 deletions
diff --git a/gold/ChangeLog b/gold/ChangeLog
index 84b00e2ae4..509db4b52d 100644
--- a/gold/ChangeLog
+++ b/gold/ChangeLog
@@ -1,3 +1,11 @@
+2013-02-11 Ian Lance Taylor <iant@google.com>
+
+ * descriptors.cc (Descriptors::close_all): New function.
+ * descriptors.h (class Descriptors): Declare close_all.
+ (close_all_descriptors): New inline function.
+ * plugin.cc: Include "descriptors.h".
+ (Plugin_manager::cleanup): Call close_all_descriptors.
+
2013-02-06 Alan Modra <amodra@gmail.com>
* README: Update coding style link.
diff --git a/gold/descriptors.cc b/gold/descriptors.cc
index f3f071e32b..b7fbaa61de 100644
--- a/gold/descriptors.cc
+++ b/gold/descriptors.cc
@@ -1,6 +1,6 @@
// descriptors.cc -- manage file descriptors for gold
-// Copyright 2008, 2009, 2010, 2011, 2012 Free Software Foundation, Inc.
+// Copyright 2008, 2009, 2010, 2011, 2012, 2013 Free Software Foundation, Inc.
// Written by Ian Lance Taylor <iant@google.com>.
// This file is part of gold.
@@ -251,6 +251,28 @@ Descriptors::close_some_descriptor()
return false;
}
+// Close all the descriptors open for reading.
+
+void
+Descriptors::close_all()
+{
+ Hold_optional_lock hl(this->lock_);
+
+ for (size_t i = 0; i < this->open_descriptors_.size(); i++)
+ {
+ Open_descriptor* pod = &this->open_descriptors_[i];
+ if (pod->name != NULL && !pod->inuse && !pod->is_write)
+ {
+ if (::close(i) < 0)
+ gold_warning(_("while closing %s: %s"), pod->name, strerror(errno));
+ pod->name = NULL;
+ pod->stack_next = -1;
+ pod->is_on_stack = false;
+ }
+ }
+ this->stack_top_ = -1;
+}
+
// The single global variable which manages descriptors.
Descriptors descriptors;
diff --git a/gold/descriptors.h b/gold/descriptors.h
index 8e154a631d..985f804574 100644
--- a/gold/descriptors.h
+++ b/gold/descriptors.h
@@ -1,6 +1,6 @@
// descriptors.h -- manage file descriptors for gold -*- C++ -*-
-// Copyright 2008, 2009 Free Software Foundation, Inc.
+// Copyright 2008, 2009, 2010, 2011, 2012, 2013 Free Software Foundation, Inc.
// Written by Ian Lance Taylor <iant@google.com>.
// This file is part of gold.
@@ -56,6 +56,10 @@ class Descriptors
void
release(int descriptor, bool permanent);
+ // Close all the descriptors open for reading.
+ void
+ close_all();
+
private:
// Information kept for a descriptor.
struct Open_descriptor
@@ -104,6 +108,10 @@ inline void
release_descriptor(int descriptor, bool permanent)
{ descriptors.release(descriptor, permanent); }
+inline void
+close_all_descriptors()
+{ descriptors.close_all(); }
+
} // End namespace gold.
#endif // !defined(GOLD_DESCRIPTORS_H)
diff --git a/gold/plugin.cc b/gold/plugin.cc
index 9176e066ef..e932c1c995 100644
--- a/gold/plugin.cc
+++ b/gold/plugin.cc
@@ -71,6 +71,7 @@ dlerror(void)
#include "target.h"
#include "readsyms.h"
#include "symtab.h"
+#include "descriptors.h"
#include "elfcpp.h"
namespace gold
@@ -697,6 +698,14 @@ Plugin_manager::layout_deferred_objects()
void
Plugin_manager::cleanup()
{
+ if (this->any_added_)
+ {
+ // If any input files were added, close all the input files.
+ // This is because the plugin may want to remove them, and on
+ // Windows you are not allowed to remove an open file.
+ close_all_descriptors();
+ }
+
for (this->current_ = this->plugins_.begin();
this->current_ != this->plugins_.end();
++this->current_)