mirror of
https://github.com/netwide-assembler/nasm.git
synced 2025-10-10 00:25:06 -04:00
out: maco64 -- Fix erroneously small write for OUT_REL4ADR
Ensure that the int64_t offset value, which ultimately comes from an int64_t value in gencode() (assemble.c:1906), is completely written to the temporary buffer, instead of merely its least significant 32 bits. Prior to this change, WRITELONG was used instead of WRITEDLONG, which resulted in add_reloc being passed an int64_t "reloff" whose least significant 32 bits were those from the aforementioned offset value, and whose most significant 32 bits were stack garbage from "mydata". This led to get_closest_section_symbol_by_offset() attempting to search for extremely large values of "offset" among the symbols in "syms", which meant that the last symbol with a matching section number would always win the symbol search. In effect, this clobbered the resultant relocation information, such that all entries would be resolved with the same symbol. Test output can be found here https://www.azabani.com/patch/2/output.txt This patch fixes http://bugzilla.nasm.us/show_bug.cgi?id=3392306 Signed-off-by: Delan Azabani <delan@azabani.com> Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>
This commit is contained in:
committed by
Cyrill Gorcunov
parent
668e58dc69
commit
5b730a197a
@@ -588,7 +588,7 @@ static void macho_output(int32_t secto, const void *data,
|
||||
|
||||
case OUT_REL4ADR:
|
||||
p = mydata;
|
||||
WRITELONG(p, *(int64_t *)data + 4 - size);
|
||||
WRITEDLONG(p, *(int64_t *)data + 4 - size);
|
||||
|
||||
if (section == secto)
|
||||
nasm_error(ERR_PANIC, "intra-section OUT_REL4ADR");
|
||||
|
||||
Reference in New Issue
Block a user