diff options
author | Cary Coutant <ccoutant@google.com> | 2013-10-11 14:12:16 -0700 |
---|---|---|
committer | Cary Coutant <ccoutant@google.com> | 2013-11-14 10:33:36 -0800 |
commit | 7cdd7d57e6fa3932eb63aa785ae0e3fa837aab20 (patch) | |
tree | 4e130cceefd5f2295ed39b574a4d43bea1567f23 /gold | |
parent | 918229560ce9db307532592047066eac0674190d (diff) | |
download | fsf-binutils-gdb-7cdd7d57e6fa3932eb63aa785ae0e3fa837aab20.zip fsf-binutils-gdb-7cdd7d57e6fa3932eb63aa785ae0e3fa837aab20.tar.gz fsf-binutils-gdb-7cdd7d57e6fa3932eb63aa785ae0e3fa837aab20.tar.bz2 |
Fix race condition while building EH frame header.
gold/
PR gold/14860
* ehframe.cc (Eh_frame_hdr::Eh_frame_hdr): Initialize lock_.
(Ehframe_hdr::set_final_data_size): Allocate a Lock.
* ehframe.h (Eh_frame_hdr::record_fde): Hold the lock while
updating fde_offsets_.
(Eh_frame_hdr::lock_): New data member.
Diffstat (limited to 'gold')
-rw-r--r-- | gold/ChangeLog | 9 | ||||
-rw-r--r-- | gold/ehframe.cc | 6 | ||||
-rw-r--r-- | gold/ehframe.h | 7 |
3 files changed, 20 insertions, 2 deletions
diff --git a/gold/ChangeLog b/gold/ChangeLog index e6910f0..5f22579 100644 --- a/gold/ChangeLog +++ b/gold/ChangeLog @@ -1,3 +1,12 @@ +2013-11-14 Cary Coutant <ccoutant@google.com> + + PR gold/14860 + * ehframe.cc (Eh_frame_hdr::Eh_frame_hdr): Initialize lock_. + (Ehframe_hdr::set_final_data_size): Allocate a Lock. + * ehframe.h (Eh_frame_hdr::record_fde): Hold the lock while + updating fde_offsets_. + (Eh_frame_hdr::lock_): New data member. + 2013-11-06 Cary Coutant <ccoutant@google.com> PR gold/15758 diff --git a/gold/ehframe.cc b/gold/ehframe.cc index 08a9ec6..be262bf 100644 --- a/gold/ehframe.cc +++ b/gold/ehframe.cc @@ -84,7 +84,8 @@ Eh_frame_hdr::Eh_frame_hdr(Output_section* eh_frame_section, eh_frame_section_(eh_frame_section), eh_frame_data_(eh_frame_data), fde_offsets_(), - any_unrecognized_eh_frame_sections_(false) + any_unrecognized_eh_frame_sections_(false), + lock_(NULL) { } @@ -102,6 +103,9 @@ Eh_frame_hdr::set_final_data_size() this->fde_offsets_.reserve(fde_count); } this->set_data_size(data_size); + // We need a lock for updating the fde_offsets_ vector while writing + // the FDEs. + this->lock_ = new Lock(); } // Write the data to the file. diff --git a/gold/ehframe.h b/gold/ehframe.h index 8aab8b8..d5ac668 100644 --- a/gold/ehframe.h +++ b/gold/ehframe.h @@ -60,7 +60,10 @@ class Eh_frame_hdr : public Output_section_data record_fde(section_offset_type fde_offset, unsigned char fde_encoding) { if (!this->any_unrecognized_eh_frame_sections_) - this->fde_offsets_.push_back(std::make_pair(fde_offset, fde_encoding)); + { + Hold_lock(*this->lock_); + this->fde_offsets_.push_back(std::make_pair(fde_offset, fde_encoding)); + } } protected: @@ -157,6 +160,8 @@ class Eh_frame_hdr : public Output_section_data // Whether we found any .eh_frame sections which we could not // process. bool any_unrecognized_eh_frame_sections_; + // Lock held while updating fde_offsets_. + Lock* lock_; }; // This class holds an FDE. |