Skip to content

Dist debugger manual signals#6487

Draft
well-mannered-goat wants to merge 10 commits into
devfrom
dist-debugger_manual_signals
Draft

Dist debugger manual signals#6487
well-mannered-goat wants to merge 10 commits into
devfrom
dist-debugger_manual_signals

Conversation

@well-mannered-goat

Copy link
Copy Markdown
Contributor

Your checklist for this pull request

  • I've read the guidelines for contributing to this repository.
  • I made sure to follow the project's coding style.
  • I've documented every RZ_API function and struct this PR changes.
  • I've added tests that prove my changes are effective (required for changes to RZ_API).
  • I've updated the Rizin book with the relevant information (if needed).
  • I've used AI tools to generate fully or partially these code changes and I'm sure the changes are not copyrighted by somebody else.

Detailed description

Ignore the s390x commit, have set that up for s390x build etc, forgot to remove it, will remove it before merging

classify the signal receive by the debuggee as internal or external
internal - due to instruction or some other cause
external - due to usage of syscall like kill to forcefully send a signal to the debuggee

if the signal is marked to be skipped
we skip the instruction causing the signal to avoid re receiving the signal

but if the signal is sent externally, do not skip an instruction

Test plan

added some tests, and removed the broken flags as in the issue, right checking it in the CI run

Closing issues

closes #5332

Comment thread librz/debug/p/native/linux/linux_debug.c Outdated
@github-actions github-actions Bot added the BSD label Jun 14, 2026
@well-mannered-goat
well-mannered-goat force-pushed the dist-debugger_manual_signals branch 5 times, most recently from d48e49e to 4846893 Compare June 23, 2026 05:14
@well-mannered-goat
well-mannered-goat force-pushed the dist-debugger_manual_signals branch from aa949c3 to 01f4be2 Compare July 5, 2026 04:28
@well-mannered-goat well-mannered-goat mentioned this pull request Jul 5, 2026
18 tasks
@well-mannered-goat
well-mannered-goat force-pushed the dist-debugger_manual_signals branch from 01f4be2 to 50b67e5 Compare July 17, 2026 04:50
@github-actions github-actions Bot added the rizin label Jul 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Signal Handler Skipping Logic Issue

2 participants