8000 linux: dso_debug: validate RDebug and LinkMap data when iterating them by schopin-mozilla · Pull Request #230 · rust-minidump/minidump-writer · GitHub
[go: up one dir, main page]

Skip to content

linux: dso_debug: validate RDebug and LinkMap data when iterating them - #230

Merged
schopin-mozilla merged 1 commit into
dso-width-agnosticfrom
dso-debug-validation
Sep 17, 2026
Merged

schopin-mozilla merged 1 commit into
dso-width-agnosticfrom
dso-debug-validation

Conversation

@schopin-mozilla
Copy link
Copy Markdown
Collaborator

This data comes from a process that has potentially corrupted memory,
it's probably a good idea to validate the data as much as possible.

We also abstract LinkMap over the Iterator trait to make it easier to
reuse it later on, e.g. for module list generation.


Stack created with GitHub Stacks CLI • Give Feedback 💬

@gabrielesvelto gabrielesvelto left a comment
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just a small nit

Comment thread src/linux/dso_debug.rs Resolved
Comment thread src/linux/dso_debug.rs Resolved
@schopin-mozilla
schopin-mozilla force-pushed the dso-debug-validation branch 3 times, most recently from 98c316c to 5d0d286 Compare September 14, 2026 11:14
This data comes from a process that has potentially corrupted memory,
it's probably a good idea to validate the data as much as possible.

We also abstract LinkMap over the Iterator trait to make it easier to
reuse it later on, e.g. for module list generation.
@schopin-mozilla
schopin-mozilla removed this pull request from stack #236 September 14, 2026 16:22
@schopin-mozilla
schopin-mozilla changed the base branch from dso-main-exec to dso-width-agnostic September 14, 2026 16:23
@schopin-mozilla
schopin-mozilla merged commit d07513e into main Sep 17, 2026
16 of 27 checks passed
@schopin-mozilla
schopin-mozilla deleted the dso-debug-validation branch September 17, 2026 11:38
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.

2 participants

0