From 89a63f1357affc67133e3dc5eb6a5bbddc7a30d6 Mon Sep 17 00:00:00 2001 From: nickc Date: Fri, 2 Jul 2004 11:12:28 +0000 Subject: [PATCH] (md_apply_fix3:BFD_RELOC_ARM_IMMEDIATE): Do not allow values which have come from undefined symbols. Always consider this fixup to have been processed as a reloc cannot be generated for it. --- gas/ChangeLog | 7 +++++++ gas/config/tc-arm.c | 15 ++++++++++++++- 2 files changed, 21 insertions(+), 1 deletion(-) diff --git a/gas/ChangeLog b/gas/ChangeLog index ab0551eb7d..42968c03a0 100644 --- a/gas/ChangeLog +++ b/gas/ChangeLog @@ -1,3 +1,10 @@ +2004-07-02 Nick Clifton + + * config/tc-arm.c (md_apply_fix3:BFD_RELOC_ARM_IMMEDIATE): Do not + allow values which have come from undefined symbols. + Always consider this fixup to have been processed as a reloc + cannot be generated for it. + 2004-07-02 Alan Modra * frags.h (struct frag): Add has_code and insn_addr fields. diff --git a/gas/config/tc-arm.c b/gas/config/tc-arm.c index 3f21c84a6c..da7fceaf42 100644 --- a/gas/config/tc-arm.c +++ b/gas/config/tc-arm.c @@ -12186,6 +12186,20 @@ md_apply_fix3 (fixP, valP, seg) switch (fixP->fx_r_type) { case BFD_RELOC_ARM_IMMEDIATE: + /* We claim that this fixup has been processed here, + even if in fact we generate an error because we do + not have a reloc for it, so tc_gen_reloc will reject it. */ + fixP->fx_done = 1; + + if (fixP->fx_addsy + && ! S_IS_DEFINED (fixP->fx_addsy)) + { + as_bad_where (fixP->fx_file, fixP->fx_line, + _("undefined symbol %s used as an immediate value"), + S_GET_NAME (fixP->fx_addsy)); + break; + } + newimm = validate_immediate (value); temp = md_chars_to_number (buf, INSN_SIZE); @@ -12202,7 +12216,6 @@ md_apply_fix3 (fixP, valP, seg) newimm |= (temp & 0xfffff000); md_number_to_chars (buf, (valueT) newimm, INSN_SIZE); - fixP->fx_done = 1; break; case BFD_RELOC_ARM_ADRL_IMMEDIATE: -- 2.11.0