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

fix: Update breakpad to version which fixes debug_id #281

Merged
merged 8 commits into from
Dec 23, 2020

Conversation

flub
Copy link
Contributor

@flub flub commented Oct 16, 2020

This version of breakpad should compute the ELF debug_identifier
correctly for big-endian minidumps processed in a little-endian
system.

This version of breakpad should compute the ELF debug_identifier
correctly for big-endian minidumps processed in a little-endian
system.
@flub flub requested a review from a team October 16, 2020 15:49
jan-auer
jan-auer previously approved these changes Oct 19, 2020
* master:
  ref: Add File/FunctionIterator and lifetimes to DebugSession (#279)
  fix: Implement new clippy advice (#280)
  fix: Add a SymbolIterator and Lifetimes to ObjectLike trait (#277)
@jan-auer jan-auer dismissed their stale review October 19, 2020 09:05

Diffs on stackwalking Linux x64 with CFI are concerning.

@Swatinem
Copy link
Member

While you are at it, can you update the directory here:

[submodule "minidump/third_party/breakpad"]

I don’t think it matters, but just to be consistent.

@Swatinem Swatinem changed the title fix: Update breakad to version which fixes debug_id fix: Update breakpad to version which fixes debug_id Oct 19, 2020
* master: (21 commits)
  fix: New nightly clippy lint (#295)
  fix(release): Do not publish examples to crates.io
  release: 8.0.0
  ref(debuginfo): Remove re-exports of error types (#300)
  meta: Update changelog
  ref(debuginfo): Switch to error with kind (#299)
  ref(minidump): Switch to error with kind (#298)
  ref(unreal): Switch to error with kind (#297)
  ref(symcache): Switch to error with kind (#296)
  feat: Rename architecture to wasm32 (#294)
  fix(debuginfo): Prefer DWARF names for Dart functions (#293)
  fix: clippy (#292)
  hack(dwarf): Skip line program sequences at 0 (#291)
  feat(wasm): support for wasm (#166)
  ci: Run rust nighlty on a schedule (#289)
  ref: Updates for rust 1.48 (#288)
  fix(debuginfo): Update dmsort to 1.0.1 to avoid panic due to UB (#287)
  ci: Use GHA instead of zeus (#286)
  ref: Introduce explicit NameMangling and better DemangleOptions (#275)
  meta: Bump all semver-major dependencies (#283)
  ...
* master:
  fix(pytest): Pin pytest 5 for debugfail customizations (#305)
Copy link
Member

@jan-auer jan-auer left a comment

Choose a reason for hiding this comment

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

LGTM. Diffs are expected due to changes in the stack scanner. Particularly, it now skips frames with no module mapping more aggressively.

@flub
Copy link
Contributor Author

flub commented Dec 23, 2020

LGTM

@jan-auer jan-auer merged commit 3d9415f into master Dec 23, 2020
@jan-auer jan-auer deleted the fix/big-endian-debugid-breakpad branch December 23, 2020 09:54
jan-auer added a commit that referenced this pull request Dec 23, 2020
* master:
  fix: Update breakpad to version which fixes debug_id (#281)
  fix(pytest): Pin pytest 5 for debugfail customizations (#305)
  fix: New nightly clippy lint (#295)
  fix(release): Do not publish examples to crates.io
  release: 8.0.0
  ref(debuginfo): Remove re-exports of error types (#300)
  meta: Update changelog
  ref(debuginfo): Switch to error with kind (#299)
  ref(minidump): Switch to error with kind (#298)
  ref(unreal): Switch to error with kind (#297)
  ref(symcache): Switch to error with kind (#296)
  feat: Rename architecture to wasm32 (#294)
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