Skip to content

Strict patch - #2996

Open
ktsakalozos wants to merge 3 commits into
masterfrom
strict
Open

Strict patch#2996
ktsakalozos wants to merge 3 commits into
masterfrom
strict

Conversation

@ktsakalozos

Copy link
Copy Markdown
Contributor

The strict patches

This was referenced Mar 21, 2022
@berkayoz
berkayoz force-pushed the strict branch 2 times, most recently from 581832b to c792a24 Compare June 6, 2022 07:29
@berkayoz
berkayoz force-pushed the strict branch 2 times, most recently from af9a253 to b9f7523 Compare June 14, 2022 14:34
@ktsakalozos
ktsakalozos force-pushed the strict branch 2 times, most recently from 63b60ef to 0b9e05c Compare June 23, 2022 09:34
@ktsakalozos
ktsakalozos force-pushed the strict branch 2 times, most recently from 6e11084 to c2febba Compare July 7, 2022 12:43
@neoaggelos
neoaggelos force-pushed the strict branch 4 times, most recently from 40d85ce to 2a473c1 Compare August 5, 2022 06:27
@ktsakalozos
ktsakalozos force-pushed the strict branch 3 times, most recently from 050b132 to 432e218 Compare August 26, 2022 03:59
@ktsakalozos
ktsakalozos force-pushed the strict branch 2 times, most recently from e3a8ab0 to 6b7eb03 Compare August 30, 2022 11:07
@ktsakalozos
ktsakalozos force-pushed the strict branch 2 times, most recently from a523188 to dec5f23 Compare December 9, 2022 08:55
@ktsakalozos
ktsakalozos force-pushed the strict branch 4 times, most recently from 8a87574 to 65c90fb Compare June 11, 2023 11:07
@ktsakalozos
ktsakalozos force-pushed the strict branch 2 times, most recently from 3ea12aa to 67ff7c9 Compare July 21, 2023 10:03
@neoaggelos
neoaggelos force-pushed the strict branch 2 times, most recently from c73c1b1 to 9da7473 Compare August 2, 2023 06:01
@sachinkumarsingh092
sachinkumarsingh092 force-pushed the strict branch 3 times, most recently from 78ea2e9 to 90f6baf Compare August 2, 2023 12:28
@ktsakalozos
ktsakalozos force-pushed the strict branch 3 times, most recently from 2ac37e6 to 8377214 Compare August 15, 2023 12:17
@neoaggelos
neoaggelos force-pushed the strict branch 2 times, most recently from 55fd479 to 1bda041 Compare November 3, 2023 09:43
@ktsakalozos
ktsakalozos force-pushed the strict branch 2 times, most recently from 0856565 to 6286626 Compare July 11, 2024 16:25
HomayoonAlimohammadi and others added 3 commits June 9, 2026 08:48
Signed-off-by: Homayoon (Hue) Alimohammadi <homayoon.alimohammadi@canonical.com>
…_init (strict) (#5552)

* fix(runc): apply AppArmor profile after finalizeNamespace in standard_init

Reorders the strict-mode runc v1.4.2 patch so the AppArmor profile change
(and NoNewPrivileges) happen after finalizeNamespace runs setuid(2),
mirroring setns_init_linux.

Because the strict patches switch the profile immediately
(aa_change_profile) instead of on exec, relabelling before setuid left the
Go runtime's sibling threads under snap.microk8s.daemon-containerd while
the calling thread moved to cri-containerd.apparmor.d. glibc's NPTL setxid
broadcast (SIGRTMIN+1) then crossed two AppArmor profiles and was denied
by AppArmor 4.x, breaking pod creation for workloads with
allowPrivilegeEscalation: false (e.g. metallb).

Doing setuid(2) while all threads share one profile keeps the broadcast
intra-profile and resolves the denials without any snapd changes.

* ci: grant security-events write permission for SARIF upload

The upload_sarifs_matrix job runs codeql-action/upload-sarif, which
requires security-events: write. With no permissions block declared the
token fell back to the restricted default and uploads failed with
"Resource not accessible by integration". Add a job-scoped permissions
block granting the minimal scopes needed (contents/actions read,
security-events write).

* ci: normalize permission comment formatting in build-snap workflow

---------

Co-authored-by: Konstantinos Tsakalozos <kos.tsakalozos@canonical.com>
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