RepoMirrors/libabigail
1
0
Fork 0
mirror of git://sourceware.org/git/libabigail.git synced 2025-02-22 16:56:57 +00:00
Code Issues Packages Projects Releases Wiki Activity

ctf-reader: Fix analyzing single kernel binaries

Browse Source 
Analyzing a single kernel binary has been broken for a long time, it
seems.  For the record, I mean this:

    $ abidw --ctf vmlinux > vmlinux.ctf.abi

Doesn't work.

This patch fixes ctf::reader::slurp_elf_info to make it not crash on
binaries that have no CTF sections because there is a .ctfa file in
the same directory.

Also, it fixes ctf::reader::read_corpus to support loading the CTF
archive for individual kernel binaries and not just for entire trees
(corpus groups).  In the case of individual kernel binaries, the
archive can be built from the CTF sections of the binary.

	* src/abg-ctf-reader.cc (reader::slurp_elf_info): Do not crash on
	a kernel binary that has no CTF section because the CTF
	information is in a .ctfa file in the same directory.
	(reader::read_corpus): Support loading a CTF archive for a single
	kernel binary.  Also, support loading the CTF archive from the CTF
	sections in the binary itself not necessarily from a .ctfa file in
	the same directory.

Signed-off-by: Dodji Seketeli <dodji@redhat.com>
This commit is contained in:
Dodji Seketeli 2024-07-10 17:14:20 +02:00
parent 57a7eb6011
commit a08a60fcd9
1 changed files with 28 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

51
src/abg-ctf-reader.cc
View File

@ -383,15 +383,6 @@ public:
elf::reader::read_corpus(status);
corpus_sptr corp = corpus();
if ((corp->get_origin() & corpus::LINUX_KERNEL_BINARY_ORIGIN)
&& corpus_group())
{
// Not finding any debug info so far is expected if we are
// building a kABI.
status &= static_cast<abigail::fe_iface::status>
(~STATUS_DEBUG_INFO_NOT_FOUND);
return;
}
if ((status & STATUS_NO_SYMBOLS_FOUND)
|| !(status & STATUS_OK))
@ -415,17 +406,29 @@ public:
}
const Elf_Scn* ctf_scn = find_ctf_section();
fill_ctf_section(ctf_scn, &ctf_sect);
if (ctf_scn)
fill_ctf_section(ctf_scn, &ctf_sect);
const Elf_Scn* symtab_scn =
elf_helpers::find_section_by_name(elf_handle(), symtab_name);
fill_ctf_section(symtab_scn, &symtab_sect);
if (symtab_scn)
fill_ctf_section(symtab_scn, &symtab_sect);
const Elf_Scn* strtab_scn =
elf_helpers::find_section_by_name(elf_handle(), strtab_name);
fill_ctf_section(strtab_scn, &strtab_sect);
if (strtab_scn)
fill_ctf_section(strtab_scn, &strtab_sect);
status |= fe_iface::STATUS_OK;
if (ctf_scn && symtab_scn && strtab_scn)
status |= fe_iface::STATUS_OK;
else if (corp->get_origin() & corpus::LINUX_KERNEL_BINARY_ORIGIN)
{
// Not finding any debug info so far is expected if we are
// building a kABI.
status &= static_cast<abigail::fe_iface::status>
(~STATUS_DEBUG_INFO_NOT_FOUND);
return;
}
}
/// Process a CTF archive and create libabigail IR for the types,
@ -690,8 +693,7 @@ public:
t.start();
int errp;
if ((corp->get_origin() & corpus::LINUX_KERNEL_BINARY_ORIGIN)
&& corpus_group())
if (corp->get_origin() & corpus::LINUX_KERNEL_BINARY_ORIGIN)
{
if (ctfa == nullptr)
{
@ -700,14 +702,17 @@ public:
ctfa = ctf_arc_open(ctfa_filename.c_str(), &errp);
}
}
else
/* Build the ctfa from the contents of the relevant ELF sections,
and process the CTF archive in the read context, if any.
Information about the types, variables, functions, etc contained
in the archive are added to the given corpus. */
if (ctfa == nullptr)
ctfa = ctf_arc_bufopen(&ctf_sect, &symtab_sect,
&strtab_sect, &errp);
/* Build the ctfa from the contents of the relevant ELF sections,
and process the CTF archive in the read context, if any.
Information about the types, variables, functions, etc contained
in the archive are added to the given corpus. */
if (ctfa == nullptr
&& ctf_sect.cts_data
&& symtab_sect.cts_data
&& strtab_sect.cts_data)
ctfa = ctf_arc_bufopen(&ctf_sect, &symtab_sect,
&strtab_sect, &errp);
if (do_log())
{