Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

i#7046: Add padding to align loadable segments to page size in the core file. #7144

Merged
merged 7 commits into from
Dec 18, 2024
Merged

i#7046: Add padding to align loadable segments to page size in the core file. #7144

Changes from 3 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
f4ffe5d
i7046: Add padding to align section data to page size.
ivankyluk Dec 17, 2024
cc9ef5b
Merge branch 'master' into i7046-add-padding-to-align-sections-on-pag…
ivankyluk Dec 17, 2024
15d7d9a
Merge branch 'master' into i7046-add-padding-to-align-sections-on-pag…
ivankyluk Dec 17, 2024
0c6a346
Incorporate review comments.
ivankyluk Dec 17, 2024
4b3529a
Merge branch 'master' into i7046-add-padding-to-align-sections-on-pag…
ivankyluk Dec 17, 2024
037117e
Increased max page size and rewrote comment for core_file_offset.
ivankyluk Dec 18, 2024
9e456d2
Remove hardcoded max page size.
ivankyluk Dec 18, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 12 additions & 5 deletions core/unix/coredump.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -484,11 +484,17 @@ os_dump_core_internal(dcontext_t *dcontext)
// of LOAD program header (number of sections minus one since shstrtab does not
ivankyluk marked this conversation as resolved.
Show resolved Hide resolved
ivankyluk marked this conversation as resolved.
Show resolved Hide resolved
ivankyluk marked this conversation as resolved.
Show resolved Hide resolved
ivankyluk marked this conversation as resolved.
Show resolved Hide resolved
// require a LOAD header).
const ELF_OFF program_header_count = section_count;
ELF_OFF file_offset = sizeof(ELF_HEADER_TYPE) +
ivankyluk marked this conversation as resolved.
Show resolved Hide resolved
sizeof(ELF_PROGRAM_HEADER_TYPE) * program_header_count +
PROGRAM_HEADER_NOTE_LENGTH;
const size_t padding = ALIGN_FORWARD(file_offset, os_page_size()) - file_offset;
ivankyluk marked this conversation as resolved.
Show resolved Hide resolved
ivankyluk marked this conversation as resolved.
Show resolved Hide resolved
file_offset = ALIGN_FORWARD(file_offset, os_page_size());

if (!write_elf_header(elf_file, /*entry_point=*/(uint64_t)mc.pc,
/*section_header_table_offset*/ sizeof(ELF_HEADER_TYPE) +
ivankyluk marked this conversation as resolved.
Show resolved Hide resolved
PROGRAM_HEADER_NOTE_LENGTH +
sizeof(ELF_PROGRAM_HEADER_TYPE) * program_header_count +
section_data_size,
padding + section_data_size,
/*flags=*/0,
/*program_header_count=*/program_header_count,
/*section_header_count=*/section_count,
Expand All @@ -509,9 +515,6 @@ os_dump_core_internal(dcontext_t *dcontext)
return false;
}
// Write loadable program segment program headers.
ELF_OFF file_offset = sizeof(ELF_HEADER_TYPE) +
sizeof(ELF_PROGRAM_HEADER_TYPE) * program_header_count +
PROGRAM_HEADER_NOTE_LENGTH;
// TODO i#7046: Merge adjacent sections with the same prot values.
// The last section is shstrtab which stores the section names and it does
// not require a LOAD program header.
Expand Down Expand Up @@ -551,6 +554,10 @@ os_dump_core_internal(dcontext_t *dcontext)
os_close(elf_file);
return false;
}
const char zero = 0;
for (int count = 0; count < padding; ++count) {
os_write(elf_file, &zero, sizeof(zero));
ivankyluk marked this conversation as resolved.
Show resolved Hide resolved
}
// Write memory content to the core dump file.
for (int section_index = 0; section_index < section_count - 1; ++section_index) {
const size_t length = section_header_info[section_index].vm_end -
Expand Down Expand Up @@ -579,7 +586,7 @@ os_dump_core_internal(dcontext_t *dcontext)
// Write section headers to the core dump file.
file_offset = sizeof(ELF_HEADER_TYPE) +
sizeof(ELF_PROGRAM_HEADER_TYPE) * program_header_count +
PROGRAM_HEADER_NOTE_LENGTH;
PROGRAM_HEADER_NOTE_LENGTH + padding;

// The section_count includes the section name section, so we need to skip
// it in the loop. The section name section is handled differently after
Expand Down
Loading