am 022885e2: Merge "Clean up all the lint cpplint can find in the dynamic linker."

* commit '022885e2fad5ef9643e05d80c6f50a2a90bf3431':
  Clean up all the lint cpplint can find in the dynamic linker.
This commit is contained in:
Elliott Hughes 2014-02-12 19:22:52 +00:00 committed by Android Git Automerger
commit bff8cdfcec
4 changed files with 61 additions and 84 deletions

View File

@ -131,13 +131,13 @@ int dladdr(const void* addr, Dl_info* info) {
info->dli_fname = si->name; info->dli_fname = si->name;
// Address at which the shared object is loaded. // Address at which the shared object is loaded.
info->dli_fbase = (void*) si->base; info->dli_fbase = reinterpret_cast<void*>(si->base);
// Determine if any symbol in the library contains the specified address. // Determine if any symbol in the library contains the specified address.
ElfW(Sym)* sym = dladdr_find_symbol(si, addr); ElfW(Sym)* sym = dladdr_find_symbol(si, addr);
if (sym != NULL) { if (sym != NULL) {
info->dli_sname = si->strtab + sym->st_name; info->dli_sname = si->strtab + sym->st_name;
info->dli_saddr = (void*)(si->load_bias + sym->st_value); info->dli_saddr = reinterpret_cast<void*>(si->load_bias + sym->st_value);
} }
return 1; return 1;

View File

@ -195,7 +195,7 @@ static void insert_soinfo_into_debug_map(soinfo* info) {
// Copy the necessary fields into the debug structure. // Copy the necessary fields into the debug structure.
link_map* map = &(info->link_map_head); link_map* map = &(info->link_map_head);
map->l_addr = info->load_bias; map->l_addr = info->load_bias;
map->l_name = (char*) info->name; map->l_name = reinterpret_cast<char*>(info->name);
map->l_ld = info->dynamic; map->l_ld = info->dynamic;
/* Stick the new library at the end of the list. /* Stick the new library at the end of the list.
@ -332,8 +332,7 @@ static soinfo* soinfo_alloc(const char* name) {
return si; return si;
} }
static void soinfo_free(soinfo* si) static void soinfo_free(soinfo* si) {
{
if (si == NULL) { if (si == NULL) {
return; return;
} }
@ -412,12 +411,10 @@ static void parse_LD_PRELOAD(const char* path) {
* *
* This function is exposed via dlfcn.cpp and libdl.so. * This function is exposed via dlfcn.cpp and libdl.so.
*/ */
_Unwind_Ptr dl_unwind_find_exidx(_Unwind_Ptr pc, int *pcount) _Unwind_Ptr dl_unwind_find_exidx(_Unwind_Ptr pc, int* pcount) {
{
soinfo *si;
unsigned addr = (unsigned)pc; unsigned addr = (unsigned)pc;
for (si = solist; si != 0; si = si->next) { for (soinfo* si = solist; si != 0; si = si->next) {
if ((addr >= si->base) && (addr < (si->base + si->size))) { if ((addr >= si->base) && (addr < (si->base + si->size))) {
*pcount = si->ARM_exidx_count; *pcount = si->ARM_exidx_count;
return (_Unwind_Ptr)si->ARM_exidx; return (_Unwind_Ptr)si->ARM_exidx;
@ -431,10 +428,7 @@ _Unwind_Ptr dl_unwind_find_exidx(_Unwind_Ptr pc, int *pcount)
/* Here, we only have to provide a callback to iterate across all the /* Here, we only have to provide a callback to iterate across all the
* loaded libraries. gcc_eh does the rest. */ * loaded libraries. gcc_eh does the rest. */
int int dl_iterate_phdr(int (*cb)(dl_phdr_info* info, size_t size, void* data), void* data) {
dl_iterate_phdr(int (*cb)(dl_phdr_info *info, size_t size, void *data),
void *data)
{
int rv = 0; int rv = 0;
for (soinfo* si = solist; si != NULL; si = si->next) { for (soinfo* si = solist; si != NULL; si = si->next) {
dl_phdr_info dl_info; dl_phdr_info dl_info;
@ -480,7 +474,7 @@ static ElfW(Sym)* soinfo_elf_lookup(soinfo* si, unsigned hash, const char* name)
} }
static unsigned elfhash(const char* _name) { static unsigned elfhash(const char* _name) {
const unsigned char* name = (const unsigned char*) _name; const unsigned char* name = reinterpret_cast<const unsigned char*>(_name);
unsigned h = 0, g; unsigned h = 0, g;
while (*name) { while (*name) {
@ -497,7 +491,6 @@ static ElfW(Sym)* soinfo_do_lookup(soinfo* si, const char* name, soinfo** lsi, s
ElfW(Sym)* s = NULL; ElfW(Sym)* s = NULL;
if (si != NULL && somain != NULL) { if (si != NULL && somain != NULL) {
/* /*
* Local scope is executable scope. Just start looking into it right away * Local scope is executable scope. Just start looking into it right away
* for the shortcut. * for the shortcut.
@ -662,19 +655,6 @@ ElfW(Sym)* dladdr_find_symbol(soinfo* si, const void* addr) {
return NULL; return NULL;
} }
#if 0
static void dump(soinfo* si)
{
ElfW(Sym)* s = si->symtab;
for (unsigned n = 0; n < si->nchain; n++) {
TRACE("%04d> %08x: %02x %04x %08x %08x %s", n, s,
s->st_info, s->st_shndx, s->st_value, s->st_size,
si->strtab + s->st_name);
s++;
}
}
#endif
static int open_library_on_path(const char* name, const char* const paths[]) { static int open_library_on_path(const char* name, const char* const paths[]) {
char buf[512]; char buf[512];
for (size_t i = 0; paths[i] != NULL; ++i) { for (size_t i = 0; paths[i] != NULL; ++i) {
@ -741,18 +721,14 @@ static soinfo* load_library(const char* name) {
return si; return si;
} }
static soinfo *find_loaded_library(const char *name) static soinfo *find_loaded_library(const char* name) {
{
soinfo *si;
const char *bname;
// TODO: don't use basename only for determining libraries // TODO: don't use basename only for determining libraries
// http://code.google.com/p/android/issues/detail?id=6670 // http://code.google.com/p/android/issues/detail?id=6670
bname = strrchr(name, '/'); const char* bname = strrchr(name, '/');
bname = bname ? bname + 1 : name; bname = bname ? bname + 1 : name;
for (si = solist; si != NULL; si = si->next) { for (soinfo* si = solist; si != NULL; si = si->next) {
if (!strcmp(bname, si->name)) { if (!strcmp(bname, si->name)) {
return si; return si;
} }
@ -870,14 +846,14 @@ static int soinfo_relocate(soinfo* si, ElfW(Rela)* rela, unsigned count, soinfo*
unsigned sym = ELFW(R_SYM)(rela->r_info); unsigned sym = ELFW(R_SYM)(rela->r_info);
ElfW(Addr) reloc = static_cast<ElfW(Addr)>(rela->r_offset + si->load_bias); ElfW(Addr) reloc = static_cast<ElfW(Addr)>(rela->r_offset + si->load_bias);
ElfW(Addr) sym_addr = 0; ElfW(Addr) sym_addr = 0;
char* sym_name = NULL; const char* sym_name = NULL;
DEBUG("Processing '%s' relocation at index %zd", si->name, idx); DEBUG("Processing '%s' relocation at index %zd", si->name, idx);
if (type == 0) { // R_*_NONE if (type == 0) { // R_*_NONE
continue; continue;
} }
if (sym != 0) { if (sym != 0) {
sym_name = (char *)(strtab + symtab[sym].st_name); sym_name = reinterpret_cast<const char*>(strtab + symtab[sym].st_name);
s = soinfo_do_lookup(si, sym_name, &lsi, needed); s = soinfo_do_lookup(si, sym_name, &lsi, needed);
if (s == NULL) { if (s == NULL) {
// We only allow an undefined symbol if this is a weak reference... // We only allow an undefined symbol if this is a weak reference...
@ -925,7 +901,7 @@ static int soinfo_relocate(soinfo* si, ElfW(Rela)* rela, unsigned count, soinfo*
break; break;
#endif #endif
default: default:
DL_ERR("unknown weak reloc type %d @ %p (%d)", type, rela, (int) (rela - start)); DL_ERR("unknown weak reloc type %d @ %p (%zu)", type, rela, idx);
return -1; return -1;
} }
} else { } else {
@ -1084,7 +1060,8 @@ static int soinfo_relocate(soinfo* si, ElfW(Rela)* rela, unsigned count, soinfo*
si->name, s->st_size, src->st_size); si->name, s->st_size, src->st_size);
return -1; return -1;
} }
memcpy((void*)reloc, (void*)(src->st_value + lsi->load_bias), src->st_size); memcpy(reinterpret_cast<void*>(reloc),
reinterpret_cast<void*>(src->st_value + lsi->load_bias), src->st_size);
} else { } else {
DL_ERR("%s R_AARCH64_COPY relocation target cannot be resolved", si->name); DL_ERR("%s R_AARCH64_COPY relocation target cannot be resolved", si->name);
return -1; return -1;
@ -1149,7 +1126,7 @@ static int soinfo_relocate(soinfo* si, ElfW(Rela)* rela, unsigned count, soinfo*
#endif #endif
default: default:
DL_ERR("unknown reloc type %d @ %p (%d)", type, rela, (int) (rela - start)); DL_ERR("unknown reloc type %d @ %p (%zu)", type, rela, idx);
return -1; return -1;
} }
} }
@ -1171,14 +1148,14 @@ static int soinfo_relocate(soinfo* si, ElfW(Rel)* rel, unsigned count, soinfo* n
unsigned sym = ELFW(R_SYM)(rel->r_info); unsigned sym = ELFW(R_SYM)(rel->r_info);
ElfW(Addr) reloc = static_cast<ElfW(Addr)>(rel->r_offset + si->load_bias); ElfW(Addr) reloc = static_cast<ElfW(Addr)>(rel->r_offset + si->load_bias);
ElfW(Addr) sym_addr = 0; ElfW(Addr) sym_addr = 0;
char* sym_name = NULL; const char* sym_name = NULL;
DEBUG("Processing '%s' relocation at index %zd", si->name, idx); DEBUG("Processing '%s' relocation at index %zd", si->name, idx);
if (type == 0) { // R_*_NONE if (type == 0) { // R_*_NONE
continue; continue;
} }
if (sym != 0) { if (sym != 0) {
sym_name = (char *)(strtab + symtab[sym].st_name); sym_name = reinterpret_cast<const char*>(strtab + symtab[sym].st_name);
s = soinfo_do_lookup(si, sym_name, &lsi, needed); s = soinfo_do_lookup(si, sym_name, &lsi, needed);
if (s == NULL) { if (s == NULL) {
// We only allow an undefined symbol if this is a weak reference... // We only allow an undefined symbol if this is a weak reference...
@ -1230,7 +1207,7 @@ static int soinfo_relocate(soinfo* si, ElfW(Rel)* rel, unsigned count, soinfo* n
// Fall through. Can't really copy if weak symbol is not found at run-time. // Fall through. Can't really copy if weak symbol is not found at run-time.
#endif #endif
default: default:
DL_ERR("unknown weak reloc type %d @ %p (%d)", type, rel, (int) (rel - start)); DL_ERR("unknown weak reloc type %d @ %p (%zu)", type, rel, idx);
return -1; return -1;
} }
} else { } else {
@ -1305,7 +1282,8 @@ static int soinfo_relocate(soinfo* si, ElfW(Rel)* rel, unsigned count, soinfo* n
si->name, s->st_size, src->st_size); si->name, s->st_size, src->st_size);
return -1; return -1;
} }
memcpy((void*)reloc, (void*)(src->st_value + lsi->load_bias), src->st_size); memcpy(reinterpret_cast<void*>(reloc),
reinterpret_cast<void*>(src->st_value + lsi->load_bias), src->st_size);
} else { } else {
DL_ERR("%s R_ARM_COPY relocation target cannot be resolved", si->name); DL_ERR("%s R_ARM_COPY relocation target cannot be resolved", si->name);
return -1; return -1;
@ -1344,9 +1322,9 @@ static int soinfo_relocate(soinfo* si, ElfW(Rel)* rel, unsigned count, soinfo* n
// We only handle the R_MIPS_NONE|R_MIPS_64|R_MIPS_REL32 case // We only handle the R_MIPS_NONE|R_MIPS_64|R_MIPS_REL32 case
if (ELF64_R_TYPE2(rel->r_info) != R_MIPS_64 || if (ELF64_R_TYPE2(rel->r_info) != R_MIPS_64 ||
ELF64_R_TYPE3(rel->r_info) != R_MIPS_NONE) { ELF64_R_TYPE3(rel->r_info) != R_MIPS_NONE) {
DL_ERR("Unexpected compound relocation type:%d type2:%d type3:%d @ %p (%d)", DL_ERR("Unexpected compound relocation type:%d type2:%d type3:%d @ %p (%zu)",
type, (unsigned)ELF64_R_TYPE2(rel->r_info), type, (unsigned)ELF64_R_TYPE2(rel->r_info),
(unsigned)ELF64_R_TYPE3(rel->r_info), rel, (int) (rel - start)); (unsigned)ELF64_R_TYPE3(rel->r_info), rel, idx);
return -1; return -1;
} }
#endif #endif
@ -1379,7 +1357,7 @@ static int soinfo_relocate(soinfo* si, ElfW(Rel)* rel, unsigned count, soinfo* n
break; break;
default: default:
DL_ERR("unknown reloc type %d @ %p (%d)", type, rel, (int) (rel - start)); DL_ERR("unknown reloc type %d @ %p (%zu)", type, rel, idx);
return -1; return -1;
} }
} }
@ -1632,16 +1610,16 @@ static bool soinfo_link_image(soinfo* si) {
d, reinterpret_cast<void*>(d->d_tag), reinterpret_cast<void*>(d->d_un.d_val)); d, reinterpret_cast<void*>(d->d_tag), reinterpret_cast<void*>(d->d_un.d_val));
switch (d->d_tag) { switch (d->d_tag) {
case DT_HASH: case DT_HASH:
si->nbucket = ((unsigned *) (base + d->d_un.d_ptr))[0]; si->nbucket = reinterpret_cast<uint32_t*>(base + d->d_un.d_ptr)[0];
si->nchain = ((unsigned *) (base + d->d_un.d_ptr))[1]; si->nchain = reinterpret_cast<uint32_t*>(base + d->d_un.d_ptr)[1];
si->bucket = (unsigned *) (base + d->d_un.d_ptr + 8); si->bucket = reinterpret_cast<uint32_t*>(base + d->d_un.d_ptr + 8);
si->chain = (unsigned *) (base + d->d_un.d_ptr + 8 + si->nbucket * 4); si->chain = reinterpret_cast<uint32_t*>(base + d->d_un.d_ptr + 8 + si->nbucket * 4);
break; break;
case DT_STRTAB: case DT_STRTAB:
si->strtab = (const char *) (base + d->d_un.d_ptr); si->strtab = reinterpret_cast<const char*>(base + d->d_un.d_ptr);
break; break;
case DT_SYMTAB: case DT_SYMTAB:
si->symtab = (ElfW(Sym)*) (base + d->d_un.d_ptr); si->symtab = reinterpret_cast<ElfW(Sym)*>(base + d->d_un.d_ptr);
break; break;
#if !defined(__LP64__) #if !defined(__LP64__)
case DT_PLTREL: case DT_PLTREL:
@ -1653,9 +1631,9 @@ static bool soinfo_link_image(soinfo* si) {
#endif #endif
case DT_JMPREL: case DT_JMPREL:
#if defined(USE_RELA) #if defined(USE_RELA)
si->plt_rela = (ElfW(Rela)*) (base + d->d_un.d_ptr); si->plt_rela = reinterpret_cast<ElfW(Rela)*>(base + d->d_un.d_ptr);
#else #else
si->plt_rel = (ElfW(Rel)*) (base + d->d_un.d_ptr); si->plt_rel = reinterpret_cast<ElfW(Rel)*>(base + d->d_un.d_ptr);
#endif #endif
break; break;
case DT_PLTRELSZ: case DT_PLTRELSZ:
@ -1686,7 +1664,7 @@ static bool soinfo_link_image(soinfo* si) {
#endif #endif
#if defined(USE_RELA) #if defined(USE_RELA)
case DT_RELA: case DT_RELA:
si->rela = (ElfW(Rela)*) (base + d->d_un.d_ptr); si->rela = reinterpret_cast<ElfW(Rela)*>(base + d->d_un.d_ptr);
break; break;
case DT_RELASZ: case DT_RELASZ:
si->rela_count = d->d_un.d_val / sizeof(ElfW(Rela)); si->rela_count = d->d_un.d_val / sizeof(ElfW(Rela));
@ -1699,7 +1677,7 @@ static bool soinfo_link_image(soinfo* si) {
return false; return false;
#else #else
case DT_REL: case DT_REL:
si->rel = (ElfW(Rel)*) (base + d->d_un.d_ptr); si->rel = reinterpret_cast<ElfW(Rel)*>(base + d->d_un.d_ptr);
break; break;
case DT_RELSZ: case DT_RELSZ:
si->rel_count = d->d_un.d_val / sizeof(ElfW(Rel)); si->rel_count = d->d_un.d_val / sizeof(ElfW(Rel));
@ -1772,7 +1750,7 @@ static bool soinfo_link_image(soinfo* si) {
case DT_MIPS_RLD_MAP: case DT_MIPS_RLD_MAP:
// Set the DT_MIPS_RLD_MAP entry to the address of _r_debug for GDB. // Set the DT_MIPS_RLD_MAP entry to the address of _r_debug for GDB.
{ {
r_debug** dp = (r_debug**) d->d_un.d_ptr; r_debug** dp = reinterpret_cast<r_debug**>(d->d_un.d_ptr);
*dp = &_r_debug; *dp = &_r_debug;
} }
break; break;
@ -1839,7 +1817,7 @@ static bool soinfo_link_image(soinfo* si) {
} }
} }
soinfo** needed = (soinfo**) alloca((1 + needed_count) * sizeof(soinfo*)); soinfo** needed = reinterpret_cast<soinfo**>(alloca((1 + needed_count) * sizeof(soinfo*)));
soinfo** pneeded = needed; soinfo** pneeded = needed;
for (ElfW(Dyn)* d = si->dynamic; d->d_tag != DT_NULL; ++d) { for (ElfW(Dyn)* d = si->dynamic; d->d_tag != DT_NULL; ++d) {
@ -2046,7 +2024,7 @@ static ElfW(Addr) __linker_init_post_relocation(KernelArgumentBlock& args, ElfW(
* expected address (wrong library or version mismatch?) * expected address (wrong library or version mismatch?)
*/ */
ElfW(Ehdr)* elf_hdr = reinterpret_cast<ElfW(Ehdr)*>(linker_base); ElfW(Ehdr)* elf_hdr = reinterpret_cast<ElfW(Ehdr)*>(linker_base);
ElfW(Phdr)* phdr = reinterpret_cast<ElfW(Phdr)*>((unsigned char*) linker_base + elf_hdr->e_phoff); ElfW(Phdr)* phdr = reinterpret_cast<ElfW(Phdr)*>(linker_base + elf_hdr->e_phoff);
phdr_table_get_dynamic_section(phdr, elf_hdr->e_phnum, linker_base, phdr_table_get_dynamic_section(phdr, elf_hdr->e_phnum, linker_base,
&linker_soinfo.dynamic, NULL, NULL); &linker_soinfo.dynamic, NULL, NULL);
insert_soinfo_into_debug_map(&linker_soinfo); insert_soinfo_into_debug_map(&linker_soinfo);
@ -2105,8 +2083,7 @@ static ElfW(Addr) __linker_init_post_relocation(KernelArgumentBlock& args, ElfW(
gettimeofday(&t1, NULL); gettimeofday(&t1, NULL);
PRINT("LINKER TIME: %s: %d microseconds", args.argv[0], (int) ( PRINT("LINKER TIME: %s: %d microseconds", args.argv[0], (int) (
(((long long)t1.tv_sec * 1000000LL) + (long long)t1.tv_usec) - (((long long)t1.tv_sec * 1000000LL) + (long long)t1.tv_usec) -
(((long long)t0.tv_sec * 1000000LL) + (long long)t0.tv_usec) (((long long)t0.tv_sec * 1000000LL) + (long long)t0.tv_usec)));
));
#endif #endif
#if STATS #if STATS
PRINT("RELO STATS: %s: %d abs, %d rel, %d copy, %d symbol", args.argv[0], PRINT("RELO STATS: %s: %d abs, %d rel, %d copy, %d symbol", args.argv[0],
@ -2159,7 +2136,7 @@ static ElfW(Addr) __linker_init_post_relocation(KernelArgumentBlock& args, ElfW(
*/ */
static ElfW(Addr) get_elf_exec_load_bias(const ElfW(Ehdr)* elf) { static ElfW(Addr) get_elf_exec_load_bias(const ElfW(Ehdr)* elf) {
ElfW(Addr) offset = elf->e_phoff; ElfW(Addr) offset = elf->e_phoff;
const ElfW(Phdr)* phdr_table = reinterpret_cast<const ElfW(Phdr)*>((char*)elf + offset); const ElfW(Phdr)* phdr_table = reinterpret_cast<const ElfW(Phdr)*>(reinterpret_cast<uintptr_t>(elf) + offset);
const ElfW(Phdr)* phdr_end = phdr_table + elf->e_phnum; const ElfW(Phdr)* phdr_end = phdr_table + elf->e_phnum;
for (const ElfW(Phdr)* phdr = phdr_table; phdr < phdr_end; phdr++) { for (const ElfW(Phdr)* phdr = phdr_table; phdr < phdr_end; phdr++) {
@ -2184,7 +2161,7 @@ extern "C" ElfW(Addr) __linker_init(void* raw_args) {
ElfW(Addr) linker_addr = args.getauxval(AT_BASE); ElfW(Addr) linker_addr = args.getauxval(AT_BASE);
ElfW(Ehdr)* elf_hdr = reinterpret_cast<ElfW(Ehdr)*>(linker_addr); ElfW(Ehdr)* elf_hdr = reinterpret_cast<ElfW(Ehdr)*>(linker_addr);
ElfW(Phdr)* phdr = reinterpret_cast<ElfW(Phdr)*>((unsigned char*) linker_addr + elf_hdr->e_phoff); ElfW(Phdr)* phdr = reinterpret_cast<ElfW(Phdr)*>(linker_addr + elf_hdr->e_phoff);
soinfo linker_so; soinfo linker_so;
memset(&linker_so, 0, sizeof(soinfo)); memset(&linker_so, 0, sizeof(soinfo));

View File

@ -337,7 +337,7 @@ bool ElfReader::LoadSegments() {
ElfW(Addr) file_length = file_end - file_page_start; ElfW(Addr) file_length = file_end - file_page_start;
if (file_length != 0) { if (file_length != 0) {
void* seg_addr = mmap((void*)seg_page_start, void* seg_addr = mmap(reinterpret_cast<void*>(seg_page_start),
file_length, file_length,
PFLAGS_TO_PROT(phdr->p_flags), PFLAGS_TO_PROT(phdr->p_flags),
MAP_FIXED|MAP_PRIVATE, MAP_FIXED|MAP_PRIVATE,
@ -352,7 +352,7 @@ bool ElfReader::LoadSegments() {
// if the segment is writable, and does not end on a page boundary, // if the segment is writable, and does not end on a page boundary,
// zero-fill it until the page limit. // zero-fill it until the page limit.
if ((phdr->p_flags & PF_W) != 0 && PAGE_OFFSET(seg_file_end) > 0) { if ((phdr->p_flags & PF_W) != 0 && PAGE_OFFSET(seg_file_end) > 0) {
memset((void*)seg_file_end, 0, PAGE_SIZE - PAGE_OFFSET(seg_file_end)); memset(reinterpret_cast<void*>(seg_file_end), 0, PAGE_SIZE - PAGE_OFFSET(seg_file_end));
} }
seg_file_end = PAGE_END(seg_file_end); seg_file_end = PAGE_END(seg_file_end);
@ -362,7 +362,7 @@ bool ElfReader::LoadSegments() {
// between them. This is done by using a private anonymous // between them. This is done by using a private anonymous
// map for all extra pages. // map for all extra pages.
if (seg_page_end > seg_file_end) { if (seg_page_end > seg_file_end) {
void* zeromap = mmap((void*)seg_file_end, void* zeromap = mmap(reinterpret_cast<void*>(seg_file_end),
seg_page_end - seg_file_end, seg_page_end - seg_file_end,
PFLAGS_TO_PROT(phdr->p_flags), PFLAGS_TO_PROT(phdr->p_flags),
MAP_FIXED|MAP_ANONYMOUS|MAP_PRIVATE, MAP_FIXED|MAP_ANONYMOUS|MAP_PRIVATE,
@ -394,7 +394,7 @@ static int _phdr_table_set_load_prot(const ElfW(Phdr)* phdr_table, size_t phdr_c
ElfW(Addr) seg_page_start = PAGE_START(phdr->p_vaddr) + load_bias; ElfW(Addr) seg_page_start = PAGE_START(phdr->p_vaddr) + load_bias;
ElfW(Addr) seg_page_end = PAGE_END(phdr->p_vaddr + phdr->p_memsz) + load_bias; ElfW(Addr) seg_page_end = PAGE_END(phdr->p_vaddr + phdr->p_memsz) + load_bias;
int ret = mprotect((void*)seg_page_start, int ret = mprotect(reinterpret_cast<void*>(seg_page_start),
seg_page_end - seg_page_start, seg_page_end - seg_page_start,
PFLAGS_TO_PROT(phdr->p_flags) | extra_prot_flags); PFLAGS_TO_PROT(phdr->p_flags) | extra_prot_flags);
if (ret < 0) { if (ret < 0) {
@ -471,7 +471,7 @@ static int _phdr_table_set_gnu_relro_prot(const ElfW(Phdr)* phdr_table, size_t p
ElfW(Addr) seg_page_start = PAGE_START(phdr->p_vaddr) + load_bias; ElfW(Addr) seg_page_start = PAGE_START(phdr->p_vaddr) + load_bias;
ElfW(Addr) seg_page_end = PAGE_END(phdr->p_vaddr + phdr->p_memsz) + load_bias; ElfW(Addr) seg_page_end = PAGE_END(phdr->p_vaddr + phdr->p_memsz) + load_bias;
int ret = mprotect((void*)seg_page_start, int ret = mprotect(reinterpret_cast<void*>(seg_page_start),
seg_page_end - seg_page_start, seg_page_end - seg_page_start,
prot_flags); prot_flags);
if (ret < 0) { if (ret < 0) {
@ -601,7 +601,7 @@ bool ElfReader::FindPhdr() {
if (phdr->p_type == PT_LOAD) { if (phdr->p_type == PT_LOAD) {
if (phdr->p_offset == 0) { if (phdr->p_offset == 0) {
ElfW(Addr) elf_addr = load_bias_ + phdr->p_vaddr; ElfW(Addr) elf_addr = load_bias_ + phdr->p_vaddr;
const ElfW(Ehdr)* ehdr = (const ElfW(Ehdr)*)(void*)elf_addr; const ElfW(Ehdr)* ehdr = reinterpret_cast<const ElfW(Ehdr)*>(elf_addr);
ElfW(Addr) offset = ehdr->e_phoff; ElfW(Addr) offset = ehdr->e_phoff;
return CheckPhdr((ElfW(Addr))ehdr + offset); return CheckPhdr((ElfW(Addr))ehdr + offset);
} }