selinux: remove the wrong comment about multithreaded process handling
authorMunehisa Kamata <kamatam@amazon.com>
Thu, 7 Dec 2023 01:33:56 +0000 (01:33 +0000)
committerPaul Moore <paul@paul-moore.com>
Thu, 7 Dec 2023 17:46:56 +0000 (12:46 -0500)
Since commit d9250dea3f89 ("SELinux: add boundary support and thread
context assignment"), SELinux has been supporting assigning per-thread
security context under a constraint and the comment was updated
accordingly. However, seems like commit d84f4f992cbd ("CRED: Inaugurate
COW credentials") accidentally brought the old comment back that doesn't
match what the code does.

Considering the ease of understanding the code, this patch just removes the
wrong comment.

Fixes: d84f4f992cbd ("CRED: Inaugurate COW credentials")
Signed-off-by: Munehisa Kamata <kamatam@amazon.com>
Signed-off-by: Paul Moore <paul@paul-moore.com>
security/selinux/hooks.c

index 855589b64641e623d8b688d8e86faaf20a54603b..863ff67e784974dd9ddce84a1f34ce709350b44a 100644 (file)
@@ -6459,7 +6459,6 @@ static int selinux_setprocattr(const char *name, void *value, size_t size)
                if (sid == 0)
                        goto abort_change;
 
-               /* Only allow single threaded processes to change context */
                if (!current_is_single_threaded()) {
                        error = security_bounded_transition(tsec->sid, sid);
                        if (error)