Skip to content

full#7

Open
PaRat07 wants to merge 1 commit into
mainfrom
feat/seccomp
Open

full#7
PaRat07 wants to merge 1 commit into
mainfrom
feat/seccomp

Conversation

@PaRat07

@PaRat07 PaRat07 commented Sep 20, 2024

Copy link
Copy Markdown
Collaborator

Теперь тестируемуму файлу запрещен доступ интернет. Это реализовано через seccomp с использованием библиотеки libseccomp.

Comment thread src/init-seccomp.cpp

scmp_filter_ctx GetFilter() {
scmp_filter_ctx ctx = seccomp_init(SCMP_ACT_ALLOW);
seccomp_rule_add(ctx, SCMP_ACT_KILL, SCMP_SYS(socket), 0);

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.

А где другие правила?

@difhel

difhel commented Sep 20, 2024

Copy link
Copy Markdown
Member

Название ПР

@difhel difhel assigned PaRat07 and unassigned difhel Sep 20, 2024
@difhel

difhel commented Sep 24, 2024

Copy link
Copy Markdown
Member

@PaRat07 а глянь, пожалуйста, что именно блокирует seccomp в ejudge, эти фильтры стоит перенести

@difhel

difhel commented Sep 26, 2024

Copy link
Copy Markdown
Member

@PaRat07

@Spotika Spotika requested a review from Copilot November 19, 2024 18:41

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

Files not reviewed (7)
  • CMakeLists.txt: Language not supported
  • include/runner/init-seccomp.h: Language not supported
  • include/runner/runner.h: Language not supported
  • main.cpp: Language not supported
  • src/init-seccomp.cpp: Language not supported
  • src/message_assert.cpp: Language not supported
  • src/runner.cpp: Language not supported

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