Skip to content

fix(minidump): Cap ELF metadata section reads - #2066

Merged
jpnurmi merged 2 commits into
masterfrom
jpnurmi/fix/elf-soname-bounds
Sep 4, 2026
Merged

fix(minidump): Cap ELF metadata section reads#2066
jpnurmi merged 2 commits into
masterfrom
jpnurmi/fix/elf-soname-bounds

Conversation

@jpnurmi

@jpnurmi jpnurmi commented Sep 4, 2026

Copy link
Copy Markdown
Collaborator

Apply the existing metadata size limit to dynamic sections and reject offsets that cannot be represented by the file API.

Close: NATIVE-221

@linear-code

linear-code Bot commented Sep 4, 2026

Copy link
Copy Markdown

NATIVE-221

Apply the existing metadata size limit to dynamic sections and reject offsets
that cannot be represented by the file API.

Close: NATIVE-221
@jpnurmi
jpnurmi force-pushed the jpnurmi/fix/elf-soname-bounds branch from 45056be to 8bf1b8e Compare September 4, 2026 13:07
@codecov

codecov Bot commented Sep 4, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 66.66667% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 74.31%. Comparing base (6223b8e) to head (e039c80).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2066      +/-   ##
==========================================
- Coverage   74.47%   74.31%   -0.16%     
==========================================
  Files         103      103              
  Lines       26713    26717       +4     
  Branches     4864     4864              
==========================================
- Hits        19894    19856      -38     
- Misses       5481     5513      +32     
- Partials     1338     1348      +10     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jpnurmi
jpnurmi merged commit 394af78 into master Sep 4, 2026
71 checks passed
@jpnurmi
jpnurmi deleted the jpnurmi/fix/elf-soname-bounds branch September 4, 2026 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants