out: maco64 -- Fix erroneously small write for OUT_REL4ADR
commit5b730a197ad343d1e3836feb49888701b9221ade
authorDelan Azabani <delan@azabani.com>
Sun, 31 May 2015 21:56:11 +0000 (1 05:56 +0800)
committerCyrill Gorcunov <gorcunov@gmail.com>
Tue, 2 Jun 2015 10:22:32 +0000 (2 13:22 +0300)
tree63d63573121fa04197d2c4d2e999c25ac434bbd6
parent668e58dc692a34ac8df54fa99c9ba28e98c338ff
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>
output/outmac64.c