Toolchain/binutils-gdb
1
0
Fork 0
forked from Imagelibrary/binutils-gdb
Code Pull Requests Activity

libctf, binutils: fix big-endian libctf archive opening

Browse Source 
The recent commit "libctf, binutils: support CTF archives like objdump"
broke opening of CTF archives on big-endian platforms.

This didn't affect anyone much before now because the linker never
emitted CTF archives because it wasn't detecting ambiguous types
properly: now it does, and this bug becomes obvious.

Fix trivial.

libctf/
	* ctf-archive.c (ctf_arc_bufopen): Endian-swap the archive magic
	number if needed.
This commit is contained in:
Nick Alcock
2020-07-01 20:10:17 +01:00
parent 62cdd7b18f
commit 4533ed564d
2 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
libctf/ChangeLog
View File

@@ -1,3 +1,8 @@
2020-07-22 Nick Alcock <nick.alcock@oracle.com>
* ctf-archive.c (ctf_arc_bufopen): Endian-swap the archive magic
number if needed.
2020-07-22 Nick Alcock <nick.alcock@oracle.com> 2020-07-22 Nick Alcock <nick.alcock@oracle.com>
* ctf-impl.h (ctf_list_splice): New. * ctf-impl.h (ctf_list_splice): New.

2
libctf/ctf-archive.c
View File

@@ -386,7 +386,7 @@ ctf_arc_bufopen (const ctf_sect_t *ctfsect, const ctf_sect_t *symsect,
ctf_file_t *fp = NULL; ctf_file_t *fp = NULL;
if (ctfsect->cts_size > sizeof (uint64_t) && if (ctfsect->cts_size > sizeof (uint64_t) &&
((*(uint64_t *) ctfsect->cts_data) == CTFA_MAGIC)) (le64toh ((*(uint64_t *) ctfsect->cts_data)) == CTFA_MAGIC))
{ {
/* The archive is mmappable, so this operation is trivial. /* The archive is mmappable, so this operation is trivial.