generic-library/bionic
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Exit normally when relocations are already packed.

Browse Source 
Bug: http://b/18051137
Change-Id: Idfffac5fe965e3cdeabe6d3b2dcd8c275c6ae5df
This commit is contained in:
Dmitriy Ivanov 2015-04-24 12:39:14 -07:00
parent 6c63ee41ac
commit b0b9338ff8
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
tools/relocation_packer/src/elf_file.cc
View File

@ -618,8 +618,8 @@ bool ElfFile<ELF>::PackTypedRelocations(std::vector<typename ELF::Rela>* relocat
typedef typename ELF::Rela Rela;
if (has_android_relocations_) {
LOG(ERROR) << "Relocation table is already packed";
return false;
LOG(INFO) << "Relocation table is already packed";
return true;
}
// If no relocations then we have nothing packable. Perhaps

4
tools/relocation_packer/src/elf_file_unittest.cc
View File

@ -103,8 +103,8 @@ template <typename ELF>
static void ProcessUnpack(FILE* relocs_so, FILE* packed_relocs_so) {
relocation_packer::ElfFile<ELF> elf_file(fileno(packed_relocs_so));
// Ensure packing fails (already packed).
EXPECT_FALSE(elf_file.PackRelocations());
// Ensure packing already packed elf-file does not fail the build.
EXPECT_TRUE(elf_file.PackRelocations());
// Unpack golden relocations, and check files are now identical.
EXPECT_TRUE(elf_file.UnpackRelocations());