hipnosuer
/
Zuma
forked from ShuriZma/suyu
1
0
Fork 0
Code Pull Requests Activity

ELF: Refactored LoadInto(..) to use memcpy, removed unnecessary code.

This commit is contained in:
bunnei 2014-06-19 17:52:15 -04:00
parent 62b444cd17
commit cedc28dcc6
1 changed files with 8 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
src/core/loader/elf.cpp
View File

@ -265,35 +265,24 @@ bool ElfReader::LoadInto(u32 vaddr) {
} else { } else {
DEBUG_LOG(MASTER_LOG, "Prerelocated executable"); DEBUG_LOG(MASTER_LOG, "Prerelocated executable");
} }
INFO_LOG(MASTER_LOG, "%i segments:", header->e_phnum); INFO_LOG(MASTER_LOG, "%i segments:", header->e_phnum);
// First pass : Get the bits into RAM // First pass : Get the bits into RAM
u32 segmentVAddr[32]; u32 segment_addr[32];
u32 baseAddress = relocate ? vaddr : 0; u32 base_addr = relocate ? vaddr : 0;
for (int i = 0; i < header->e_phnum; i++) { for (int i = 0; i < header->e_phnum; i++) {
Elf32_Phdr *p = segments + i; Elf32_Phdr *p = segments + i;
INFO_LOG(MASTER_LOG, "Type: %i Vaddr: %08x Filesz: %i Memsz: %i ", p->p_type, p->p_vaddr,
INFO_LOG(MASTER_LOG, "Type: %i Vaddr: %08x Filesz: %i Memsz: %i ", p->p_type, p->p_vaddr, p->p_filesz, p->p_memsz); p->p_filesz, p->p_memsz);
if (p->p_type == PT_LOAD) { if (p->p_type == PT_LOAD) {
segmentVAddr[i] = baseAddress + p->p_vaddr; segment_addr[i] = base_addr + p->p_vaddr;
u32 writeAddr = segmentVAddr[i]; memcpy(Memory::GetPointer(segment_addr[i]), GetSegmentPtr(i), p->p_filesz);
INFO_LOG(MASTER_LOG, "Loadable Segment Copied to %08x, size %08x", segment_addr[i],
const u8 *src = GetSegmentPtr(i); p->p_memsz);
u8 *dst = Memory::GetPointer(writeAddr);
u32 srcSize = p->p_filesz;
u32 dstSize = p->p_memsz;
u32 *s = (u32*)src;
u32 *d = (u32*)dst;
for (int j = 0; j < (int)(srcSize + 3) / 4; j++) {
*d++ = (*s++);
}
INFO_LOG(MASTER_LOG, "Loadable Segment Copied to %08x, size %08x", writeAddr, p->p_memsz);
} }
} }
INFO_LOG(MASTER_LOG, "Done loading."); INFO_LOG(MASTER_LOG, "Done loading.");
return true; return true;
} }