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.

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

Workaround crash in Microsoft.DiaSymReader.Native #68990

Merged
merged 4 commits into from
May 16, 2022

Conversation

mikem8361
Copy link
Member

Issue: #59077

The fix is to check if the module has a portable PDB in the debug directory
and use the well-tested managed portable PDB source/line number code.

Add support for in-memory embedded PDBs.

@ghost
Copy link

ghost commented May 6, 2022

Tagging subscribers to this area: @tommcdon
See info in area-owners.md if you want to be subscribed.

Issue Details

Issue: #59077

The fix is to check if the module has a portable PDB in the debug directory
and use the well-tested managed portable PDB source/line number code.

Add support for in-memory embedded PDBs.

Author: mikem8361
Assignees: mikem8361
Labels:

area-System.Diagnostics

Milestone: -

@mikem8361 mikem8361 force-pushed the diasymreader branch 2 times, most recently from db67a21 to f3b9e0f Compare May 11, 2022 23:24
@mikem8361 mikem8361 requested a review from noahfalk May 13, 2022 16:33
mikem8361 added 2 commits May 16, 2022 11:30
Issue: dotnet#59077

The fix is to check if the module has a portable PDB in the debug directory
and use the well-tested managed portable PDB source/line number code.

Add support for in-memory embedded PDBs.
@mikem8361
Copy link
Member Author

Can I get a review? Thanks.

src/coreclr/vm/debugdebugger.cpp Show resolved Hide resolved
Copy link
Member

@noahfalk noahfalk left a comment

Choose a reason for hiding this comment

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

LGTM, a few comments would help : )

src/coreclr/vm/debugdebugger.cpp Show resolved Hide resolved
src/coreclr/vm/debugdebugger.cpp Show resolved Hide resolved
@mikem8361 mikem8361 merged commit 901e494 into dotnet:main May 16, 2022
@mikem8361 mikem8361 deleted the diasymreader branch May 16, 2022 22:10
@ghost ghost locked as resolved and limited conversation to collaborators Jun 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants