mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2024-12-28 16:53:49 +00:00
54a6e6bbf3
Currently, a sandbox process is not restricted to sending a signal (e.g. SIGKILL) to a process outside the sandbox environment. The ability to send a signal for a sandboxed process should be scoped the same way abstract UNIX sockets are scoped. Therefore, we extend the "scoped" field in a ruleset with LANDLOCK_SCOPE_SIGNAL to specify that a ruleset will deny sending any signal from within a sandbox process to its parent (i.e. any parent sandbox or non-sandboxed processes). This patch adds file_set_fowner and file_free_security hooks to set and release a pointer to the file owner's domain. This pointer, fown_domain in landlock_file_security will be used in file_send_sigiotask to check if the process can send a signal. The ruleset_with_unknown_scope test is updated to support LANDLOCK_SCOPE_SIGNAL. This depends on two new changes: - commit1934b21261
("file: reclaim 24 bytes from f_owner"): replace container_of(fown, struct file, f_owner) with fown->file . - commit26f204380a
("fs: Fix file_set_fowner LSM hook inconsistencies"): lock before calling the hook. Signed-off-by: Tahera Fahimi <fahimitahera@gmail.com> Closes: https://github.com/landlock-lsm/linux/issues/8 Link: https://lore.kernel.org/r/df2b4f880a2ed3042992689a793ea0951f6798a5.1725657727.git.fahimitahera@gmail.com [mic: Update landlock_get_current_domain()'s return type, improve and fix locking in hook_file_set_fowner(), simplify and fix sleepable call and locking issue in hook_file_send_sigiotask() and rebase on the latest VFS tree, simplify hook_task_kill() and quickly return when not sandboxed, improve comments, rename LANDLOCK_SCOPED_SIGNAL] Co-developed-by: Mickaël Salaün <mic@digikod.net> Signed-off-by: Mickaël Salaün <mic@digikod.net>
103 lines
3.2 KiB
C
103 lines
3.2 KiB
C
/* SPDX-License-Identifier: GPL-2.0-only */
|
|
/*
|
|
* Landlock LSM - Filesystem management and hooks
|
|
*
|
|
* Copyright © 2017-2020 Mickaël Salaün <mic@digikod.net>
|
|
* Copyright © 2018-2020 ANSSI
|
|
*/
|
|
|
|
#ifndef _SECURITY_LANDLOCK_FS_H
|
|
#define _SECURITY_LANDLOCK_FS_H
|
|
|
|
#include <linux/fs.h>
|
|
#include <linux/init.h>
|
|
#include <linux/rcupdate.h>
|
|
|
|
#include "ruleset.h"
|
|
#include "setup.h"
|
|
|
|
/**
|
|
* struct landlock_inode_security - Inode security blob
|
|
*
|
|
* Enable to reference a &struct landlock_object tied to an inode (i.e.
|
|
* underlying object).
|
|
*/
|
|
struct landlock_inode_security {
|
|
/**
|
|
* @object: Weak pointer to an allocated object. All assignments of a
|
|
* new object are protected by the underlying inode->i_lock. However,
|
|
* atomically disassociating @object from the inode is only protected
|
|
* by @object->lock, from the time @object's usage refcount drops to
|
|
* zero to the time this pointer is nulled out (cf. release_inode() and
|
|
* hook_sb_delete()). Indeed, such disassociation doesn't require
|
|
* inode->i_lock thanks to the careful rcu_access_pointer() check
|
|
* performed by get_inode_object().
|
|
*/
|
|
struct landlock_object __rcu *object;
|
|
};
|
|
|
|
/**
|
|
* struct landlock_file_security - File security blob
|
|
*
|
|
* This information is populated when opening a file in hook_file_open, and
|
|
* tracks the relevant Landlock access rights that were available at the time
|
|
* of opening the file. Other LSM hooks use these rights in order to authorize
|
|
* operations on already opened files.
|
|
*/
|
|
struct landlock_file_security {
|
|
/**
|
|
* @allowed_access: Access rights that were available at the time of
|
|
* opening the file. This is not necessarily the full set of access
|
|
* rights available at that time, but it's the necessary subset as
|
|
* needed to authorize later operations on the open file.
|
|
*/
|
|
access_mask_t allowed_access;
|
|
/**
|
|
* @fown_domain: Domain of the task that set the PID that may receive a
|
|
* signal e.g., SIGURG when writing MSG_OOB to the related socket.
|
|
* This pointer is protected by the related file->f_owner->lock, as for
|
|
* fown_struct's members: pid, uid, and euid.
|
|
*/
|
|
struct landlock_ruleset *fown_domain;
|
|
};
|
|
|
|
/**
|
|
* struct landlock_superblock_security - Superblock security blob
|
|
*
|
|
* Enable hook_sb_delete() to wait for concurrent calls to release_inode().
|
|
*/
|
|
struct landlock_superblock_security {
|
|
/**
|
|
* @inode_refs: Number of pending inodes (from this superblock) that
|
|
* are being released by release_inode().
|
|
* Cf. struct super_block->s_fsnotify_inode_refs .
|
|
*/
|
|
atomic_long_t inode_refs;
|
|
};
|
|
|
|
static inline struct landlock_file_security *
|
|
landlock_file(const struct file *const file)
|
|
{
|
|
return file->f_security + landlock_blob_sizes.lbs_file;
|
|
}
|
|
|
|
static inline struct landlock_inode_security *
|
|
landlock_inode(const struct inode *const inode)
|
|
{
|
|
return inode->i_security + landlock_blob_sizes.lbs_inode;
|
|
}
|
|
|
|
static inline struct landlock_superblock_security *
|
|
landlock_superblock(const struct super_block *const superblock)
|
|
{
|
|
return superblock->s_security + landlock_blob_sizes.lbs_superblock;
|
|
}
|
|
|
|
__init void landlock_add_fs_hooks(void);
|
|
|
|
int landlock_append_fs_rule(struct landlock_ruleset *const ruleset,
|
|
const struct path *const path,
|
|
access_mask_t access_hierarchy);
|
|
|
|
#endif /* _SECURITY_LANDLOCK_FS_H */
|