Skip to content

Remote corruption - #175

Merged
mubes merged 10 commits into
mainfrom
remote_corruption
Nov 8, 2026
Merged

mubes merged 10 commits into
mainfrom
remote_corruption

Conversation

@mubes

@mubes mubes commented Nov 8, 2026

Copy link
Copy Markdown
Collaborator

Fix slow remote client handling

lunakiller and others added 10 commits February 17, 2026 12:41
Both the project source root issue and the incorrect calloc argument
order were fixed in upstream libdwarf, the related patches were removed.
Ported patch 0002 to 2.3.2, and introduced a new patch to fix another
unused variable in libdwarf.
A previous refactor to remove another protocol mode accidentally made
ETM return a signal indicating argument parsing failure instead of using
the protocol.

Fix by not returning early. Experiments show the protocol works
perfectly still.
The source loader accidentally ate the character after each newline,
rather than the newline itself. This led to consecutive lines
disappearing, and lines sometimes losing their first character. The
former screwed up the association between lines in the debug info and
the file displayed by orbmortem.

Fix by zeroing the newline character itself. The character is already
consumed according to the `l` and `r` variables; it just needs changing
to `\0`.
@mubes
mubes force-pushed the remote_corruption branch from 5479635 to aac1507 Compare November 8, 2026 22:35
@mubes
mubes merged commit 769db68 into main Nov 8, 2026
6 checks passed
@mubes
mubes deleted the remote_corruption branch November 8, 2026 23: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.

4 participants