* elf32-i386.c (i386_opcode16): Delete.

(elf_i386_check_tls_transition): Use memcmp to compare contents.
	* elf64-x86-64.c (x86_64_opcode16, x86_64_opcode32): Delete.
	(elf_x86_64_check_tls_transition): Use memcmp to compare contents.
This commit is contained in:
Alan Modra
2011-10-19 23:02:50 +00:00
parent 0319698bac
commit fa289a5fde
3 changed files with 20 additions and 37 deletions

View File

@ -1,3 +1,10 @@
2011-10-20 Alan Modra <amodra@gmail.com>
* elf32-i386.c (i386_opcode16): Delete.
(elf_i386_check_tls_transition): Use memcmp to compare contents.
* elf64-x86-64.c (x86_64_opcode16, x86_64_opcode32): Delete.
(elf_x86_64_check_tls_transition): Use memcmp to compare contents.
2011-10-19 Alan Modra <amodra@gmail.com> 2011-10-19 Alan Modra <amodra@gmail.com>
PR ld/13177 PR ld/13177

View File

@ -1094,13 +1094,6 @@ elf_i386_copy_indirect_symbol (struct bfd_link_info *info,
_bfd_elf_link_hash_copy_indirect (info, dir, ind); _bfd_elf_link_hash_copy_indirect (info, dir, ind);
} }
typedef union
{
unsigned char c[2];
uint16_t i;
}
i386_opcode16;
/* Return TRUE if the TLS access code sequence support transition /* Return TRUE if the TLS access code sequence support transition
from R_TYPE. */ from R_TYPE. */
@ -1271,8 +1264,8 @@ elf_i386_check_tls_transition (bfd *abfd, asection *sec,
if (offset + 2 <= sec->size) if (offset + 2 <= sec->size)
{ {
/* Make sure that it's a call *x@tlsdesc(%rax). */ /* Make sure that it's a call *x@tlsdesc(%rax). */
static i386_opcode16 call = { { 0xff, 0x10 } }; static const unsigned char call[] = { 0xff, 0x10 };
return bfd_get_16 (abfd, contents + offset) == call.i; return memcmp (contents + offset, call, 2) == 0;
} }
return FALSE; return FALSE;

View File

@ -1007,20 +1007,6 @@ elf64_x86_64_elf_object_p (bfd *abfd)
return TRUE; return TRUE;
} }
typedef union
{
unsigned char c[2];
uint16_t i;
}
x86_64_opcode16;
typedef union
{
unsigned char c[4];
uint32_t i;
}
x86_64_opcode32;
/* Return TRUE if the TLS access code sequence support transition /* Return TRUE if the TLS access code sequence support transition
from R_TYPE. */ from R_TYPE. */
@ -1076,24 +1062,23 @@ elf_x86_64_check_tls_transition (bfd *abfd,
.word 0x6666; rex64; call __tls_get_addr .word 0x6666; rex64; call __tls_get_addr
can transit to different access model. */ can transit to different access model. */
static x86_64_opcode32 call = { { 0x66, 0x66, 0x48, 0xe8 } }; static const unsigned char call[] = { 0x66, 0x66, 0x48, 0xe8 };
static const unsigned char leaq[] = { 0x66, 0x48, 0x8d, 0x3d };
if ((offset + 12) > sec->size if ((offset + 12) > sec->size
|| bfd_get_32 (abfd, contents + offset + 4) != call.i) || memcmp (contents + offset + 4, call, 4) != 0)
return FALSE; return FALSE;
if (ABI_64_P (abfd)) if (ABI_64_P (abfd))
{ {
static x86_64_opcode32 leaq = { { 0x66, 0x48, 0x8d, 0x3d } };
if (offset < 4 if (offset < 4
|| bfd_get_32 (abfd, contents + offset - 4) != leaq.i) || memcmp (contents + offset - 4, leaq, 4) != 0)
return FALSE; return FALSE;
} }
else else
{ {
static x86_64_opcode16 lea = { { 0x8d, 0x3d } };
if (offset < 3 if (offset < 3
|| bfd_get_8 (abfd, contents + offset - 3) != 0x48 || memcmp (contents + offset - 3, leaq + 1, 3) != 0)
|| bfd_get_16 (abfd, contents + offset - 2) != lea.i)
return FALSE; return FALSE;
} }
} }
@ -1104,15 +1089,13 @@ elf_x86_64_check_tls_transition (bfd *abfd,
call __tls_get_addr call __tls_get_addr
can transit to different access model. */ can transit to different access model. */
static x86_64_opcode32 ld = { { 0x48, 0x8d, 0x3d, 0xe8 } }; static const unsigned char lea[] = { 0x48, 0x8d, 0x3d };
x86_64_opcode32 op;
if (offset < 3 || (offset + 9) > sec->size) if (offset < 3 || (offset + 9) > sec->size)
return FALSE; return FALSE;
op.i = bfd_get_32 (abfd, contents + offset - 3); if (memcmp (contents + offset - 3, lea, 3) != 0
op.c[3] = bfd_get_8 (abfd, contents + offset + 4); || 0xe8 != *(contents + offset + 4))
if (op.i != ld.i)
return FALSE; return FALSE;
} }
@ -1191,8 +1174,8 @@ elf_x86_64_check_tls_transition (bfd *abfd,
if (offset + 2 <= sec->size) if (offset + 2 <= sec->size)
{ {
/* Make sure that it's a call *x@tlsdesc(%rax). */ /* Make sure that it's a call *x@tlsdesc(%rax). */
static x86_64_opcode16 call = { { 0xff, 0x10 } }; static const unsigned char call[] = { 0xff, 0x10 };
return bfd_get_16 (abfd, contents + offset) == call.i; return memcmp (contents + offset, call, 2) == 0;
} }
return FALSE; return FALSE;