Skip to content

fix: bad seccomp detection - #1208

Open
AntennaeVY wants to merge 4 commits into
DMOJ:masterfrom
AntennaeVY:master
Open

fix: bad seccomp detection#1208
AntennaeVY wants to merge 4 commits into
DMOJ:masterfrom
AntennaeVY:master

Conversation

@AntennaeVY

Copy link
Copy Markdown

Previous method failed for kernel versions with aditional characters like 6.6.72+

Feel free to left any comment or improvement.

Improved bad seccomp detection
@dmoj-build

Copy link
Copy Markdown
Collaborator

Can one of the admins verify this patch?

@Xyene Xyene left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, thanks!

Comment thread dmoj/cptbox/tracer.py Outdated

@Xyene Xyene left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Whoops, CI doesn't pass.

@AntennaeVY
AntennaeVY requested a review from Xyene June 15, 2025 04:14
@AntennaeVY

Copy link
Copy Markdown
Author

Whoops, CI doesn't pass.

I fixed though

@AntennaeVY

Copy link
Copy Markdown
Author

Fixed formatting errors

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