Skip to content

fix(native): Fix smaller findings for Linux minidump writer#1694

Open
mujacica wants to merge 7 commits intomasterfrom
fix/native-minidump-linux
Open

fix(native): Fix smaller findings for Linux minidump writer#1694
mujacica wants to merge 7 commits intomasterfrom
fix/native-minidump-linux

Conversation

@mujacica
Copy link
Copy Markdown
Contributor

@mujacica mujacica commented May 4, 2026

  • Native/Linux: correct MD_LINUX_MAPS stream type (was tagged as MD_LINUX_AUXV).
  • Native/Linux: drop non-ELF mappings (e.g. /dev/shm/*, (deleted) files) from the minidump module list.
  • Native/Linux: merge non-contiguous mappings of the same shared library into a single module, and use the offset==0 mapping as base_of_image. Fixes duplicate ld-linux entries that confused some debuggers (notably Windows LLDB) reading Linux ARM64 minidumps.
  • Native/Linux: log when uname() is blocked (sandbox/seccomp) and fall back to /proc/sys/kernel/osrelease for the OS version.
  • Native/Linux: emit LinuxAuxv, LinuxCpuInfo, LinuxLsbRelease, LinuxCmdLine, LinuxEnviron, and LinuxDsoDebug streams alongside the existing set. LLDB needs LinuxAuxv and LinuxDsoDebug to identify the dynamic loader and enumerate loaded shared libraries; without them, opening a minidump in LLDB on Linux would only recover one frame per thread.
  • Native/Linux: replay each thread's stack memory descriptor into MemoryListStream. Previously stack bytes were only referenced from the per-thread record, so debuggers that look up memory by virtual address (LLDB) could not read the stack and unwinding stopped at frame 0 even when eh_frame was available.

@mujacica mujacica force-pushed the fix/native-minidump-linux branch from b8a9319 to 5fd69fb Compare May 4, 2026 10:18
@mujacica mujacica marked this pull request as ready for review May 4, 2026 21:54
Comment thread src/backends/native/minidump/sentry_minidump_linux.c Outdated
Comment thread src/backends/native/minidump/sentry_minidump_linux.c Outdated
Comment thread src/backends/native/minidump/sentry_minidump_format.h
Comment thread src/backends/native/minidump/sentry_minidump_linux.c
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 6fca3dd. Configure here.

Comment thread src/backends/native/minidump/sentry_minidump_macos.c
Comment thread src/backends/native/minidump/sentry_minidump_linux.c
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.

1 participant