dnrops
/
patchelf
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix issue #66 by ignoring the first section header when sorting, and not overwriting NOBITS entries.

This commit is contained in:
Ezra Cooper 2018-06-21 11:07:35 -07:00
parent 27ffe8ae87
commit 52ab908394
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/patchelf.cc
View File

@ -475,7 +475,7 @@ void ElfFile<ElfFileParamNames>::sortShdrs()
/* Sort the sections by offset. */ /* Sort the sections by offset. */
CompShdr comp; CompShdr comp;
comp.elfFile = this; comp.elfFile = this;
sort(shdrs.begin(), shdrs.end(), comp); sort(shdrs.begin() + 1, shdrs.end(), comp);
/* Restore the sh_link mappings. */ /* Restore the sh_link mappings. */
for (unsigned int i = 1; i < rdi(hdr->e_shnum); ++i) for (unsigned int i = 1; i < rdi(hdr->e_shnum); ++i)
@ -642,7 +642,8 @@ void ElfFile<ElfFileParamNames>::writeReplacedSections(Elf_Off & curOff,
for (auto & i : replacedSections) { for (auto & i : replacedSections) {
std::string sectionName = i.first; std::string sectionName = i.first;
Elf_Shdr & shdr = findSection(sectionName); Elf_Shdr & shdr = findSection(sectionName);
memset(contents + rdi(shdr.sh_offset), 'X', rdi(shdr.sh_size)); if (shdr.sh_type != SHT_NOBITS)
memset(contents + rdi(shdr.sh_offset), 'X', rdi(shdr.sh_size));
} }
for (auto & i : replacedSections) { for (auto & i : replacedSections) {