LineageOS/android_development
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge "Check machine arch before detecting relocation type"

Browse Source
This commit is contained in:
Treehugger Robot
2018-08-03 01:51:06 +00:00
committed by Gerrit Code Review
parent 77b844cb98 0a93212704
commit 36efe44516
1 changed files with 42 additions and 24 deletions
Download Patch File Download Diff File Expand all files Collapse all files

66
vndk/tools/vtable-dumper/elf_handling.cpp
View File

@@ -18,18 +18,24 @@
#include <cxxabi.h>
using llvm::ELF::ELFDATA2MSB;
using llvm::ELF::EM_386;
using llvm::ELF::EM_X86_64;
using llvm::ELF::EM_ARM;
using llvm::ELF::EM_AARCH64;
using llvm::ELF::EM_MIPS;
using llvm::ELF::R_AARCH64_ABS64;
using llvm::ELF::R_AARCH64_RELATIVE;
using llvm::ELF::R_ARM_ABS32;
using llvm::ELF::R_ARM_RELATIVE;
using llvm::ELF::R_386_32;
using llvm::ELF::R_386_RELATIVE;
using llvm::ELF::R_X86_64_64;
using llvm::ELF::R_X86_64_RELATIVE;
using llvm::ELF::R_ARM_ABS32;
using llvm::ELF::R_ARM_RELATIVE;
using llvm::ELF::R_AARCH64_ABS64;
using llvm::ELF::R_AARCH64_RELATIVE;
using llvm::ELF::R_MIPS_64;
using llvm::ELF::R_MIPS_REL32;
using llvm::ELF::R_MIPS_NONE;
using llvm::ELF::SHT_PROGBITS;
using llvm::ELF::SHT_REL;
using llvm::ELF::SHT_RELA;
@@ -60,7 +66,6 @@ static std::string demangle(const std::string &MangledName) {
return "";
}
SharedObject::~SharedObject() {}
template <typename ELFT>
static std::unique_ptr<SharedObject> createELFSharedObject(
@@ -68,7 +73,7 @@ static std::unique_ptr<SharedObject> createELFSharedObject(
return make_unique<ELFSharedObject<ELFT>>(Objfile);
}
static std::unique_ptr<SharedObject>createELFObjFile(const ObjectFile *Obj) {
static std::unique_ptr<SharedObject> createELFObjFile(const ObjectFile *Obj) {
if (const ELF32LEObjectFile *Objfile = dyn_cast<ELF32LEObjectFile>(Obj))
return createELFSharedObject(Objfile);
if (const ELF32BEObjectFile *Objfile = dyn_cast<ELF32BEObjectFile>(Obj))
@@ -81,6 +86,8 @@ static std::unique_ptr<SharedObject>createELFObjFile(const ObjectFile *Obj) {
return nullptr;
}
SharedObject::~SharedObject() {}
std::unique_ptr<SharedObject> SharedObject::create(const ObjectFile *Obj) {
std::unique_ptr<SharedObject> res(createELFObjFile(Obj));
if (res && res->getVTables()) {
@@ -246,28 +253,39 @@ void ELFSharedObject<ELFT>::relocateSym(
relativeRelocation(Relocation, Section, Vtablep);
}
} else {
switch(Relocation.getType()) {
case R_AARCH64_RELATIVE:
case R_X86_64_RELATIVE:
case R_ARM_RELATIVE:
{
// The return value is ignored since failure to relocate
// does not mean a fatal error. It might be that the dynsym /
// symbol-table does not have enough information to get the
// symbol name. Like-wise for absolute relocations.
relativeRelocation(Relocation, Section, Vtablep);
bool IsAbsoluteRelocation = false;
bool IsRelativeRelocation = false;
uint32_t RelocationType = Relocation.getType();
switch (ElfHeader->e_machine) {
case EM_ARM:
IsAbsoluteRelocation = RelocationType == R_ARM_ABS32;
IsRelativeRelocation = RelocationType == R_ARM_RELATIVE;
break;
}
case R_AARCH64_ABS64:
case R_X86_64_64:
case R_ARM_ABS32:
{
absoluteRelocation(Relocation, Vtablep);
case EM_AARCH64:
IsAbsoluteRelocation = RelocationType == R_AARCH64_ABS64;
IsRelativeRelocation = RelocationType == R_AARCH64_RELATIVE;
break;
case EM_386:
IsAbsoluteRelocation = RelocationType == R_386_32;
IsRelativeRelocation = RelocationType == R_386_RELATIVE;
break;
case EM_X86_64:
IsAbsoluteRelocation = RelocationType == R_X86_64_64;
IsRelativeRelocation = RelocationType == R_X86_64_RELATIVE;
break;
}
default:
break;
}
// The return value is ignored since failure to relocate
// does not mean a fatal error. It might be that the dynsym /
// symbol-table does not have enough information to get the
// symbol name. Like-wise for absolute relocations.
if (IsAbsoluteRelocation) {
absoluteRelocation(Relocation, Vtablep);
}
if (IsRelativeRelocation) {
relativeRelocation(Relocation, Section, Vtablep);
}
}
}