From 3d0064a95d413e7d19ff9c7f6db53fefe44a697b Mon Sep 17 00:00:00 2001 From: Cary Coutant Date: Wed, 14 Jan 2015 10:30:14 -0800 Subject: [PATCH] Fix a bug in resolving HI16/LO16 relocation pairs for MIPS. 2015-01-14 Sasa Stankovic gold/ * mips.cc (reloc_high): Add r_sym. (Mips_relocate_functions::relhi16): Add r_sym parameter. Pass r_sym to reloc_high constructor. (Mips_relocate_functions::relgot16_local): Likewise. (Mips_relocate_functions::rello16): Add r_sym parameter. Use r_sym and r_type to decide whether LO16 matches HI16. (Target_mips::Relocate::relocate): Pass r_sym to calls to relhi16, rello16 and relgot16_local. --- gold/ChangeLog | 11 +++++++++++ gold/mips.cc | 30 +++++++++++++++++------------- 2 files changed, 28 insertions(+), 13 deletions(-) diff --git a/gold/ChangeLog b/gold/ChangeLog index 1e73aa8effb..9a52a3e86d1 100644 --- a/gold/ChangeLog +++ b/gold/ChangeLog @@ -1,3 +1,14 @@ +2015-01-14 Sasa Stankovic + + * mips.cc (reloc_high): Add r_sym. + (Mips_relocate_functions::relhi16): Add r_sym parameter. Pass r_sym to + reloc_high constructor. + (Mips_relocate_functions::relgot16_local): Likewise. + (Mips_relocate_functions::rello16): Add r_sym parameter. Use r_sym and + r_type to decide whether LO16 matches HI16. + (Target_mips::Relocate::relocate): Pass r_sym to calls to relhi16, + rello16 and relgot16_local. + 2015-01-09 Cary Coutant * layout.cc (Layout::set_segment_offsets): Don't align start of segment diff --git a/gold/mips.cc b/gold/mips.cc index 97cb4924a65..acf76cffaec 100644 --- a/gold/mips.cc +++ b/gold/mips.cc @@ -3761,11 +3761,11 @@ struct reloc_high reloc_high(unsigned char* _view, const Mips_relobj* _object, const Symbol_value* _psymval, Mips_address _addend, - unsigned int _r_type, bool _extract_addend, + unsigned int _r_type, unsigned int _r_sym, bool _extract_addend, Mips_address _address = 0, bool _gp_disp = false) : view(_view), object(_object), psymval(_psymval), addend(_addend), - r_type(_r_type), extract_addend(_extract_addend), address(_address), - gp_disp(_gp_disp) + r_type(_r_type), r_sym(_r_sym), extract_addend(_extract_addend), + address(_address), gp_disp(_gp_disp) { } unsigned char* view; @@ -3773,6 +3773,7 @@ struct reloc_high const Symbol_value* psymval; Mips_address addend; unsigned int r_type; + unsigned int r_sym; bool extract_addend; Mips_address address; bool gp_disp; @@ -4266,11 +4267,12 @@ class Mips_relocate_functions : public Relocate_functions relhi16(unsigned char* view, const Mips_relobj* object, const Symbol_value* psymval, Mips_address addend, Mips_address address, bool gp_disp, unsigned int r_type, - bool extract_addend) + unsigned int r_sym, bool extract_addend) { // Record the relocation. It will be resolved when we find lo16 part. hi16_relocs.push_back(reloc_high(view, object, psymval, - addend, r_type, extract_addend, address, gp_disp)); + addend, r_type, r_sym, extract_addend, address, + gp_disp)); return This::STATUS_OKAY; } @@ -4331,11 +4333,11 @@ class Mips_relocate_functions : public Relocate_functions relgot16_local(unsigned char* view, const Mips_relobj* object, const Symbol_value* psymval, Mips_address addend_a, - bool extract_addend, unsigned int r_type) + bool extract_addend, unsigned int r_type, unsigned int r_sym) { // Record the relocation. It will be resolved when we find lo16 part. got16_relocs.push_back(reloc_high(view, object, psymval, - addend_a, r_type, extract_addend)); + addend_a, r_type, r_sym, extract_addend)); return This::STATUS_OKAY; } @@ -4377,7 +4379,7 @@ class Mips_relocate_functions : public Relocate_functions const Mips_relobj* object, const Symbol_value* psymval, Mips_address addend_a, bool extract_addend, Mips_address address, bool is_gp_disp, - unsigned int r_type) + unsigned int r_type, unsigned int r_sym) { mips_reloc_unshuffle(view, r_type, false); Valtype32* wv = reinterpret_cast(view); @@ -4392,7 +4394,8 @@ class Mips_relocate_functions : public Relocate_functions while (it != hi16_relocs.end()) { reloc_high hi16 = *it; - if (hi16.psymval->value(hi16.object, 0) == psymval->value(object, 0)) + if (hi16.r_sym == r_sym + && is_matching_lo16_reloc(hi16.r_type, r_type)) { if (do_relhi16(hi16.view, hi16.object, hi16.psymval, hi16.addend, hi16.address, hi16.gp_disp, hi16.r_type, @@ -4411,7 +4414,8 @@ class Mips_relocate_functions : public Relocate_functions while (it2 != got16_relocs.end()) { reloc_high got16 = *it2; - if (got16.psymval->value(got16.object, 0) == psymval->value(object, 0)) + if (got16.r_sym == r_sym + && is_matching_lo16_reloc(got16.r_type, r_type)) { if (do_relgot16_local(got16.view, got16.object, got16.psymval, got16.addend, got16.r_type, @@ -9908,7 +9912,7 @@ Target_mips::Relocate::relocate( case elfcpp::R_MIPS16_HI16: case elfcpp::R_MICROMIPS_HI16: reloc_status = Reloc_funcs::relhi16(view, object, psymval, r_addend, - address, gp_disp, r_type, + address, gp_disp, r_type, r_sym, extract_addend); break; @@ -9918,7 +9922,7 @@ Target_mips::Relocate::relocate( case elfcpp::R_MICROMIPS_HI0_LO16: reloc_status = Reloc_funcs::rello16(target, view, object, psymval, r_addend, extract_addend, address, - gp_disp, r_type); + gp_disp, r_type, r_sym); break; case elfcpp::R_MIPS_LITERAL: @@ -10033,7 +10037,7 @@ Target_mips::Relocate::relocate( else reloc_status = Reloc_funcs::relgot16_local(view, object, psymval, r_addend, extract_addend, - r_type); + r_type, r_sym); update_got_entry = changed_symbol_value; break; -- 2.30.2