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

RUMM-2025 Filter out unrecognized trailing stack frame in error.stack #794

Merged

Conversation

ncreated
Copy link
Member

What and why?

📦 This PR aims at improving the symbolicated error.stack by removing trailing ??? frame (if present).

Going from this:
Screenshot 2022-03-29 at 14 09 37

to this:
Screenshot 2022-03-29 at 14 35 32

The root cause of ??? is lack of library base address resolved for the frame. This makes its symbolication impossible. This line describes the very entry point to the code execution (even before main), so nothing actionable. To keep the symbolicated stack clean and avoid false positive symbolication errors, we truncate it.

How?

Only the last frame is dropped and only if it couldn't be symbolicated.

Review checklist

  • Feature or bugfix MUST have appropriate tests (unit, integration)
  • Make sure each commit and the PR mention the Issue number or JIRA reference
  • Add CHANGELOG entry for user facing change.

(only if the frame has no library base address read).
@ncreated ncreated requested a review from a team as a code owner March 29, 2022 13:51
@ncreated ncreated self-assigned this Mar 29, 2022
Copy link
Member

@maxep maxep left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@ncreated ncreated merged commit 770206d into develop Mar 29, 2022
@ncreated ncreated deleted the ncreated/RUMM-2025-filter-out-stack-frames-with-addr-0x00 branch March 29, 2022 15:06
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