]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commit
COFF: correct function auxiliary symbol data clearing
authorJan Beulich <jbeulich@suse.com>
Fri, 2 May 2025 08:08:19 +0000 (10:08 +0200)
committerJan Beulich <jbeulich@suse.com>
Fri, 2 May 2025 08:08:19 +0000 (10:08 +0200)
commit89911183bd070b9e21a520db3a5213a7d1910bf6
tree03f790596c5677ae0d0ebf524464b28b48a1f730
parentea9c4d157496d68a489f1d9e3a623d1dd9788192
COFF: correct function auxiliary symbol data clearing

It's unclear why the array part of the union was used there, when we're
dealing with a function. Originally, when 32-bit hosts and targets were
prevailing, the memset() in question ended up clearing the entire x_fcn,
while for 64-bit hosts/targets only x_lnnoptr would have been cleared.
Then a2c7ca15a560 ("Use stdint types in coff internal_auxent") made
things consistent, but imo in the wrong direction (and likely
unintentionally). Go back to what apparently was meant originally, using
the correct part of the union now.
gas/config/obj-coff.c