Skip to content

Don't allow switching before recording a negative syscall's entry - #4132

Open
Keno wants to merge 1 commit into
rr-debugger:masterfrom
ChronitonAI:negative-syscall-switch
Open

Keno wants to merge 1 commit into
rr-debugger:masterfrom
ChronitonAI:negative-syscall-switch

Conversation

@Keno

@Keno Keno commented Oct 20, 2026

Copy link
Copy Markdown
Member

The kernel skips a syscall with a negative number after its seccomp stop, and rr records the syscall's entry at the next record_step. Allowing a switch in between let other threads' events be recorded first, even if they depended on what the thread did before the syscall, so replay could diverge. Keep switching prevented there, as for seccomp-induced errno and trap results.

Test: negative_syscall_switch

The kernel skips a syscall with a negative number after its seccomp
stop, and rr records the syscall's entry at the next record_step.
Allowing a switch in between let other threads' events be recorded
first, even if they depended on what the thread did before the
syscall, so replay could diverge. Keep switching prevented there, as
for seccomp-induced errno and trap results.

Test: negative_syscall_switch

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@rocallahan

Copy link
Copy Markdown
Collaborator

Looks good but needs merge conflicts to be resolved

This branch has not been deployed

No deployments
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