From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp-8fa9.mail.infomaniak.ch (smtp-8fa9.mail.infomaniak.ch [83.166.143.169]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 90BDD3C5DA0 for ; Wed, 8 Apr 2026 19:21:25 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=83.166.143.169 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775676090; cv=none; b=IjgFsqqtHIKkRwMBiYM/l79YZYP9g6Yq5bzGJv6BSyFPGKTnY3TSv8QocweRXSp6v1E+4MB4Oa2NA0mn1epN3Eg2jrYli5frWaM0etTENxMpWSVFaHeIekBsw7nZGYZYIJN7uINI3dzj+On5DUgJLX+vASAWw/m4fBjJ6ajnfTk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1775676090; c=relaxed/simple; bh=w/vChAtlWYNJS+fM3MqBkAXddxe84quORPtoobVD8Bg=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=heNEmfaCjV2xG7J0ULkr+Au1cmfmh2QjuueD+OjsVi9K7KRqouRFWlcSVJzw1Q5wSzNZEzQXK83LBfvnU3E/UDSgIOGy7bMes1rQ0h/XNhiFw23S3oFlnMahsgcu6Ymr2UJ/6y3s5fQMWunvCSRpdBpkIEvjEkgH+YgLJX8dwFQ= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=digikod.net; spf=pass smtp.mailfrom=digikod.net; dkim=pass (1024-bit key) header.d=digikod.net header.i=@digikod.net header.b=hH9ITMQf; arc=none smtp.client-ip=83.166.143.169 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=digikod.net Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=digikod.net Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=digikod.net header.i=@digikod.net header.b="hH9ITMQf" Received: from smtp-4-0000.mail.infomaniak.ch (unknown [IPv6:2001:1600:7:10::a6b]) by smtp-4-3000.mail.infomaniak.ch (Postfix) with ESMTPS id 4frXwt2Cmsz13BT; Wed, 8 Apr 2026 21:21:18 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=digikod.net; s=20191114; t=1775676078; bh=lqgJbO6VKY1s5BuLIOCqFqlJGru+qWSHakL+QAEcm6k=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=hH9ITMQfuNU8tov8EpppnqI3Nr4J/WBjxXU3lvalSA4MvJBCurY8jg4Q4+CAO+4V1 Zc0asqtOkKf7mmlin7bvrkX//+F+b9DRyeCldNUkiNopDSG0EtUpYM5Yc1lZqr0afS qevN2L1tk5dh2Kv+K/bFAsAzyOJu4SPVXRqPLZO8= Received: from unknown by smtp-4-0000.mail.infomaniak.ch (Postfix) with ESMTPA id 4frXwr54JhzWF3; Wed, 8 Apr 2026 21:21:16 +0200 (CEST) Date: Wed, 8 Apr 2026 21:21:11 +0200 From: =?utf-8?Q?Micka=C3=ABl_Sala=C3=BCn?= To: Justin Suess Cc: andrii@kernel.org, ast@kernel.org, bpf@vger.kernel.org, brauner@kernel.org, daniel@iogearbox.net, eddyz87@gmail.com, fred@cloudflare.com, gnoack@google.com, jack@suse.cz, jmorris@namei.org, john.fastabend@gmail.com, kees@kernel.org, kpsingh@kernel.org, linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, linux-security-module@vger.kernel.org, m@maowtm.org, martin.lau@linux.dev, paul@paul-moore.com Subject: Re: [RFC PATCH 00/20] BPF interface for applying Landlock rulesets Message-ID: <20260408.ainu5Chohnge@digikod.net> References: <20260408.ong9Eshe0omu@digikod.net> <20260408171030.4083129-1-utilityemal77@gmail.com> Precedence: bulk X-Mailing-List: linux-fsdevel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20260408171030.4083129-1-utilityemal77@gmail.com> X-Infomaniak-Routing: alpha On Wed, Apr 08, 2026 at 01:10:28PM -0400, Justin Suess wrote: > > Add a flag LANDLOCK_RESTRICT_SELF_NO_NEW_PRIVS, which executes > task_set_no_new_privs on the current credentials, but only if > the process lacks the CAP_SYS_ADMIN capability. > > While this operation is redundant for code running from userspace > (indeed callers may achieve the same logic by calling > prctl w/ PR_SET_NO_NEW_PRIVS), this flag enables callers without access > to the syscall abi (defined in subsequent patches) to restrict processes > from gaining additional capabilities. This is important to ensure that > consumers can meet the task_no_new_privs || CAP_SYS_ADMIN invariant > enforced by Landlock without having syscall access. > > This is done by hooking bprm_committing_creds along with a > landlock_cred_security flag to indicate that the next execution should > task_set_no_new_privs if the process doesn't possess CAP_SYS_ADMIN. This > is done to ensure that task_set_no_new_privs is being done past the > point of no return. > > Cc: Mickaël Salaün > Signed-off-by: Justin Suess > --- > > On Wed, Apr 08, 2026 at 02:00:00 -0000, Mickaël Salaün wrote: > > > Points of Feedback > > > === > > > > > > First, the new set_nnp_on_point_of_no_return field in struct linux_binprm. > > > This field was needed to request that task_set_no_new_privs be set during an > > > execution, but only after the execution has proceeded beyond the point of no > > > return. I couldn't find a way to express this semantic without adding a new > > > bitfield to struct linux_binprm and a conditional in fs/exec.c. Please see > > > patch 2. > > > What about using security_bprm_committing_creds()? > > Good idea. Definitely cleaner. > > Something like this? Then dropping the "execve: Add set_nnp_on_point_of_no_return" > commit. > > This adds a bitfield to the landlock_cred_security struct to indicate that the flag > should be set on the next exec(s). > > include/uapi/linux/landlock.h | 14 ++++++++++++++ > security/landlock/cred.c | 13 +++++++++++++ > security/landlock/cred.h | 7 +++++++ > security/landlock/limits.h | 2 +- > security/landlock/ruleset.c | 15 ++++++++++++--- > security/landlock/syscalls.c | 5 +++++ > 6 files changed, 52 insertions(+), 4 deletions(-) > > diff --git a/include/uapi/linux/landlock.h b/include/uapi/linux/landlock.h > index f88fa1f68b77..edd9d9a7f60e 100644 > --- a/include/uapi/linux/landlock.h > +++ b/include/uapi/linux/landlock.h > @@ -129,12 +129,26 @@ struct landlock_ruleset_attr { > * > * If the calling thread is running with no_new_privs, this operation > * enables no_new_privs on the sibling threads as well. > + * > + * %LANDLOCK_RESTRICT_SELF_NO_NEW_PRIVS > + * Sets no_new_privs on the calling thread before applying the Landlock domain. > + * This flag is useful for convenience as well as for applying a ruleset from > + * an outside context (e.g BPF). This flag only has an effect on when both > + * no_new_privs isn't already set and the caller doesn't possess CAP_SYS_ADMIN. > + * > + * This flag has slightly different behavior when used from BPF. Instead of > + * setting no_new_privs on the current task, it sets a flag on the bprm so that > + * no_new_privs is set on the task at exec point-of-no-return. This guarantees > + * that the current execution is unaffected, and may escalate as usual until the > + * next exec, but the resulting task cannot gain more privileges through later > + * exec transitions. > */ > /* clang-format off */ > #define LANDLOCK_RESTRICT_SELF_LOG_SAME_EXEC_OFF (1U << 0) > #define LANDLOCK_RESTRICT_SELF_LOG_NEW_EXEC_ON (1U << 1) > #define LANDLOCK_RESTRICT_SELF_LOG_SUBDOMAINS_OFF (1U << 2) > #define LANDLOCK_RESTRICT_SELF_TSYNC (1U << 3) > +#define LANDLOCK_RESTRICT_SELF_NO_NEW_PRIVS (1U << 4) > /* clang-format on */ > > /** > diff --git a/security/landlock/cred.c b/security/landlock/cred.c > index 0cb3edde4d18..bcc9b716916f 100644 > --- a/security/landlock/cred.c > +++ b/security/landlock/cred.c > @@ -43,6 +43,18 @@ static void hook_cred_free(struct cred *const cred) > landlock_put_ruleset_deferred(dom); > } > > +static void hook_bprm_committing_creds(const struct linux_binprm *bprm) > +{ > + struct landlock_cred_security *const llcred = landlock_cred(bprm->cred); > + > + if (llcred->set_nnp_on_committing_creds && > + !ns_capable_noaudit(current_user_ns(), CAP_SYS_ADMIN)) { If asked by the caller, NNP must be set, whatever the capabilities of the task. > + task_set_no_new_privs(current); > + /* Don't need to set it again for subsequent execution. */ > + llcred->set_nnp_on_committing_creds = false; > + } Thinking more about it, it would make more sense to add another flag to enforce restriction on the next exec. This new cred bit would then be generic and enforce both NNP (if set) and the domain once we know the execution is ok. That should also bring the required plumbing to create the domain at syscall (or kfunc) time and handle memory allocation issue there, but only enforce it at exec time with security_bprm_committing_creds() (without any possible error). > +} > + > #ifdef CONFIG_AUDIT > > static int hook_bprm_creds_for_exec(struct linux_binprm *const bprm) > @@ -55,6 +67,7 @@ static int hook_bprm_creds_for_exec(struct linux_binprm *const bprm) > #endif /* CONFIG_AUDIT */ > > static struct security_hook_list landlock_hooks[] __ro_after_init = { > + LSM_HOOK_INIT(bprm_committing_creds, hook_bprm_committing_creds), > LSM_HOOK_INIT(cred_prepare, hook_cred_prepare), > LSM_HOOK_INIT(cred_transfer, hook_cred_transfer), > LSM_HOOK_INIT(cred_free, hook_cred_free), > diff --git a/security/landlock/cred.h b/security/landlock/cred.h > index c10a06727eb1..7ec6dd12ebc3 100644 > --- a/security/landlock/cred.h > +++ b/security/landlock/cred.h > @@ -49,6 +49,13 @@ struct landlock_cred_security { > * not require a current domain. > */ > u8 log_subdomains_off : 1; > + /** > + * @set_nnp_on_committing_creds: Set if the domain should set NO_NEW_PRIVS on the > + * execution past the point of no return in security_bprm_committing_creds(). > + * This is not a hierarchy configuration because the nnp state is inherited by > + * exec and doesn't need further configuration. > + */ > + u8 set_nnp_on_committing_creds : 1; > #endif /* CONFIG_AUDIT */ > } __packed; > > diff --git a/security/landlock/limits.h b/security/landlock/limits.h > index eb584f47288d..d298086a4180 100644 > --- a/security/landlock/limits.h > +++ b/security/landlock/limits.h > @@ -31,7 +31,7 @@ > #define LANDLOCK_MASK_SCOPE ((LANDLOCK_LAST_SCOPE << 1) - 1) > #define LANDLOCK_NUM_SCOPE __const_hweight64(LANDLOCK_MASK_SCOPE) > > -#define LANDLOCK_LAST_RESTRICT_SELF LANDLOCK_RESTRICT_SELF_TSYNC > +#define LANDLOCK_LAST_RESTRICT_SELF LANDLOCK_RESTRICT_SELF_NO_NEW_PRIVS > #define LANDLOCK_MASK_RESTRICT_SELF ((LANDLOCK_LAST_RESTRICT_SELF << 1) - 1) > > /* clang-format on */ > diff --git a/security/landlock/ruleset.c b/security/landlock/ruleset.c > index 1d6fa74f2a52..ad0bd5994ec5 100644 > --- a/security/landlock/ruleset.c > +++ b/security/landlock/ruleset.c > @@ -121,11 +121,13 @@ int landlock_restrict_cred_precheck(const __u32 flags, > > /* > * Similar checks as for seccomp(2), except that an -EPERM may be > - * returned. > + * returned, or no_new_privs may be set by the caller via > + * LANDLOCK_RESTRICT_SELF_NO_NEW_PRIVS. > */ > if (!task_no_new_privs(current) && > !ns_capable_noaudit(current_user_ns(), CAP_SYS_ADMIN)) { > - return -EPERM; > + if (!(flags & LANDLOCK_RESTRICT_SELF_NO_NEW_PRIVS)) > + return -EPERM; > } > > if (flags & ~LANDLOCK_MASK_RESTRICT_SELF) > @@ -140,7 +142,7 @@ int landlock_restrict_cred(struct cred *const cred, > { > struct landlock_cred_security *new_llcred; > bool __maybe_unused log_same_exec, log_new_exec, log_subdomains, > - prev_log_subdomains; > + prev_log_subdomains, set_nnp_on_committing_creds; > > /* > * It is allowed to set LANDLOCK_RESTRICT_SELF_LOG_SUBDOMAINS_OFF without > @@ -157,6 +159,12 @@ int landlock_restrict_cred(struct cred *const cred, > log_new_exec = !!(flags & LANDLOCK_RESTRICT_SELF_LOG_NEW_EXEC_ON); > /* Translates "off" flag to boolean. */ > log_subdomains = !(flags & LANDLOCK_RESTRICT_SELF_LOG_SUBDOMAINS_OFF); > + /* > + * Translates "on" flag to boolean. This flag is not inherited by exec, > + * but the resulting nnp state is. > + */ > + set_nnp_on_committing_creds = > + !!(flags & LANDLOCK_RESTRICT_SELF_NO_NEW_PRIVS); > > new_llcred = landlock_cred(cred); > > @@ -165,6 +173,7 @@ int landlock_restrict_cred(struct cred *const cred, > new_llcred->log_subdomains_off = !prev_log_subdomains || > !log_subdomains; > #endif /* CONFIG_AUDIT */ > + new_llcred->set_nnp_on_committing_creds = set_nnp_on_committing_creds; > > /* > * The only case when a ruleset may not be set is if > diff --git a/security/landlock/syscalls.c b/security/landlock/syscalls.c > index c6c7be7698a2..f3520c764360 100644 > --- a/security/landlock/syscalls.c > +++ b/security/landlock/syscalls.c > @@ -397,6 +397,7 @@ SYSCALL_DEFINE4(landlock_add_rule, const int, ruleset_fd, > * - %LANDLOCK_RESTRICT_SELF_LOG_NEW_EXEC_ON > * - %LANDLOCK_RESTRICT_SELF_LOG_SUBDOMAINS_OFF > * - %LANDLOCK_RESTRICT_SELF_TSYNC > + * - %LANDLOCK_RESTRICT_SELF_NO_NEW_PRIVS > * > * This system call enforces a Landlock ruleset on the current thread. > * Enforcing a ruleset requires that the task has %CAP_SYS_ADMIN in its > @@ -450,6 +451,10 @@ SYSCALL_DEFINE2(landlock_restrict_self, const int, ruleset_fd, const __u32, > if (!new_cred) > return -ENOMEM; > > + if (flags & LANDLOCK_RESTRICT_SELF_NO_NEW_PRIVS && > + !ns_capable_noaudit(current_user_ns(), CAP_SYS_ADMIN)) > + task_set_no_new_privs(current); > + > err = landlock_restrict_cred(new_cred, ruleset, flags); > if (err) { > abort_creds(new_cred); > -- > 2.53.0 > >