Ensure we use calculate_first_tls_offset, even if the main program doesn't

have TLS program header. This is needed on architectures with Variant I
tls, that is arm, arm64, mips, and powerpc. These place the thread control
block at the start of the buffer and, without this, this data may be
trashed.

This appears to not be an issue on mips or powerpc as they include a second
adjustment to move the thread local data, however this is on arm64 (with a
future change to fix placing this data), and should be on arm. I am unable
to trigger this on arm, even after changing the code to move the data
around to make it more likely to be hit. This is most likely because my
tests didn't use the variable in offset 0.

Reviewed by:	kib
MFC after:	1 week
Sponsored by:	ABT Systems Ltd
This commit is contained in:
Andrew Turner 2015-09-01 15:43:56 +00:00
parent 104dac63f3
commit 878165d2ef
Notes: svn2git 2020-12-20 02:59:44 +00:00
svn path=/head/; revision=287369

View File

@ -4611,7 +4611,7 @@ allocate_tls_offset(Obj_Entry *obj)
return true;
}
if (obj->tlsindex == 1)
if (tls_last_offset == 0)
off = calculate_first_tls_offset(obj->tlssize, obj->tlsalign);
else
off = calculate_tls_offset(tls_last_offset, tls_last_size,