mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-08-27 19:29:37 +00:00
eventfs: Save ownership and mode
commit28e12c09f5
upstream Now that inodes and dentries are created on the fly, they are also reclaimed on memory pressure. Since the ownership and file mode are saved in the inode, if they are freed, any changes to the ownership and mode will be lost. To counter this, if the user changes the permissions or ownership, save them, and when creating the inodes again, restore those changes. Link: https://lkml.kernel.org/r/20231101172649.691841445@goodmis.org Cc: stable@vger.kernel.org Cc: Ajay Kaher <akaher@vmware.com> Cc: Masami Hiramatsu <mhiramat@kernel.org> Cc: Mark Rutland <mark.rutland@arm.com> Cc: Andrew Morton <akpm@linux-foundation.org> Fixes:6394044955
("eventfs: Implement eventfs lookup, read, open functions") Reviewed-by: Masami Hiramatsu (Google) <mhiramat@kernel.org> Signed-off-by: Steven Rostedt (Google) <rostedt@goodmis.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
fa18a8a053
commit
9aaee3eebc
1 changed files with 91 additions and 16 deletions
|
@ -40,6 +40,8 @@ struct eventfs_inode {
|
|||
* @data: something that the caller will want to get to later on
|
||||
* @is_freed: Flag set if the eventfs is on its way to be freed
|
||||
* @mode: the permission that the file or directory should have
|
||||
* @uid: saved uid if changed
|
||||
* @gid: saved gid if changed
|
||||
*/
|
||||
struct eventfs_file {
|
||||
const char *name;
|
||||
|
@ -61,11 +63,22 @@ struct eventfs_file {
|
|||
void *data;
|
||||
unsigned int is_freed:1;
|
||||
unsigned int mode:31;
|
||||
kuid_t uid;
|
||||
kgid_t gid;
|
||||
};
|
||||
|
||||
static DEFINE_MUTEX(eventfs_mutex);
|
||||
DEFINE_STATIC_SRCU(eventfs_srcu);
|
||||
|
||||
/* Mode is unsigned short, use the upper bits for flags */
|
||||
enum {
|
||||
EVENTFS_SAVE_MODE = BIT(16),
|
||||
EVENTFS_SAVE_UID = BIT(17),
|
||||
EVENTFS_SAVE_GID = BIT(18),
|
||||
};
|
||||
|
||||
#define EVENTFS_MODE_MASK (EVENTFS_SAVE_MODE - 1)
|
||||
|
||||
static struct dentry *eventfs_root_lookup(struct inode *dir,
|
||||
struct dentry *dentry,
|
||||
unsigned int flags);
|
||||
|
@ -73,8 +86,54 @@ static int dcache_dir_open_wrapper(struct inode *inode, struct file *file);
|
|||
static int dcache_readdir_wrapper(struct file *file, struct dir_context *ctx);
|
||||
static int eventfs_release(struct inode *inode, struct file *file);
|
||||
|
||||
static void update_attr(struct eventfs_file *ef, struct iattr *iattr)
|
||||
{
|
||||
unsigned int ia_valid = iattr->ia_valid;
|
||||
|
||||
if (ia_valid & ATTR_MODE) {
|
||||
ef->mode = (ef->mode & ~EVENTFS_MODE_MASK) |
|
||||
(iattr->ia_mode & EVENTFS_MODE_MASK) |
|
||||
EVENTFS_SAVE_MODE;
|
||||
}
|
||||
if (ia_valid & ATTR_UID) {
|
||||
ef->mode |= EVENTFS_SAVE_UID;
|
||||
ef->uid = iattr->ia_uid;
|
||||
}
|
||||
if (ia_valid & ATTR_GID) {
|
||||
ef->mode |= EVENTFS_SAVE_GID;
|
||||
ef->gid = iattr->ia_gid;
|
||||
}
|
||||
}
|
||||
|
||||
static int eventfs_set_attr(struct mnt_idmap *idmap, struct dentry *dentry,
|
||||
struct iattr *iattr)
|
||||
{
|
||||
struct eventfs_file *ef;
|
||||
int ret;
|
||||
|
||||
mutex_lock(&eventfs_mutex);
|
||||
ef = dentry->d_fsdata;
|
||||
/* The LSB is set when the eventfs_inode is being freed */
|
||||
if (((unsigned long)ef & 1UL) || ef->is_freed) {
|
||||
/* Do not allow changes if the event is about to be removed. */
|
||||
mutex_unlock(&eventfs_mutex);
|
||||
return -ENODEV;
|
||||
}
|
||||
|
||||
ret = simple_setattr(idmap, dentry, iattr);
|
||||
if (!ret)
|
||||
update_attr(ef, iattr);
|
||||
mutex_unlock(&eventfs_mutex);
|
||||
return ret;
|
||||
}
|
||||
|
||||
static const struct inode_operations eventfs_root_dir_inode_operations = {
|
||||
.lookup = eventfs_root_lookup,
|
||||
.setattr = eventfs_set_attr,
|
||||
};
|
||||
|
||||
static const struct inode_operations eventfs_file_inode_operations = {
|
||||
.setattr = eventfs_set_attr,
|
||||
};
|
||||
|
||||
static const struct file_operations eventfs_file_operations = {
|
||||
|
@ -85,10 +144,20 @@ static const struct file_operations eventfs_file_operations = {
|
|||
.release = eventfs_release,
|
||||
};
|
||||
|
||||
static void update_inode_attr(struct inode *inode, struct eventfs_file *ef)
|
||||
{
|
||||
inode->i_mode = ef->mode & EVENTFS_MODE_MASK;
|
||||
|
||||
if (ef->mode & EVENTFS_SAVE_UID)
|
||||
inode->i_uid = ef->uid;
|
||||
|
||||
if (ef->mode & EVENTFS_SAVE_GID)
|
||||
inode->i_gid = ef->gid;
|
||||
}
|
||||
|
||||
/**
|
||||
* create_file - create a file in the tracefs filesystem
|
||||
* @name: the name of the file to create.
|
||||
* @mode: the permission that the file should have.
|
||||
* @ef: the eventfs_file
|
||||
* @parent: parent dentry for this file.
|
||||
* @data: something that the caller will want to get to later on.
|
||||
* @fop: struct file_operations that should be used for this file.
|
||||
|
@ -104,7 +173,7 @@ static const struct file_operations eventfs_file_operations = {
|
|||
* If tracefs is not enabled in the kernel, the value -%ENODEV will be
|
||||
* returned.
|
||||
*/
|
||||
static struct dentry *create_file(const char *name, umode_t mode,
|
||||
static struct dentry *create_file(struct eventfs_file *ef,
|
||||
struct dentry *parent, void *data,
|
||||
const struct file_operations *fop)
|
||||
{
|
||||
|
@ -112,13 +181,13 @@ static struct dentry *create_file(const char *name, umode_t mode,
|
|||
struct dentry *dentry;
|
||||
struct inode *inode;
|
||||
|
||||
if (!(mode & S_IFMT))
|
||||
mode |= S_IFREG;
|
||||
if (!(ef->mode & S_IFMT))
|
||||
ef->mode |= S_IFREG;
|
||||
|
||||
if (WARN_ON_ONCE(!S_ISREG(mode)))
|
||||
if (WARN_ON_ONCE(!S_ISREG(ef->mode)))
|
||||
return NULL;
|
||||
|
||||
dentry = eventfs_start_creating(name, parent);
|
||||
dentry = eventfs_start_creating(ef->name, parent);
|
||||
|
||||
if (IS_ERR(dentry))
|
||||
return dentry;
|
||||
|
@ -127,7 +196,10 @@ static struct dentry *create_file(const char *name, umode_t mode,
|
|||
if (unlikely(!inode))
|
||||
return eventfs_failed_creating(dentry);
|
||||
|
||||
inode->i_mode = mode;
|
||||
/* If the user updated the directory's attributes, use them */
|
||||
update_inode_attr(inode, ef);
|
||||
|
||||
inode->i_op = &eventfs_file_inode_operations;
|
||||
inode->i_fop = fop;
|
||||
inode->i_private = data;
|
||||
|
||||
|
@ -140,7 +212,7 @@ static struct dentry *create_file(const char *name, umode_t mode,
|
|||
|
||||
/**
|
||||
* create_dir - create a dir in the tracefs filesystem
|
||||
* @name: the name of the file to create.
|
||||
* @ei: the eventfs_inode that represents the directory to create
|
||||
* @parent: parent dentry for this file.
|
||||
* @data: something that the caller will want to get to later on.
|
||||
*
|
||||
|
@ -155,13 +227,14 @@ static struct dentry *create_file(const char *name, umode_t mode,
|
|||
* If tracefs is not enabled in the kernel, the value -%ENODEV will be
|
||||
* returned.
|
||||
*/
|
||||
static struct dentry *create_dir(const char *name, struct dentry *parent, void *data)
|
||||
static struct dentry *create_dir(struct eventfs_file *ef,
|
||||
struct dentry *parent, void *data)
|
||||
{
|
||||
struct tracefs_inode *ti;
|
||||
struct dentry *dentry;
|
||||
struct inode *inode;
|
||||
|
||||
dentry = eventfs_start_creating(name, parent);
|
||||
dentry = eventfs_start_creating(ef->name, parent);
|
||||
if (IS_ERR(dentry))
|
||||
return dentry;
|
||||
|
||||
|
@ -169,7 +242,8 @@ static struct dentry *create_dir(const char *name, struct dentry *parent, void *
|
|||
if (unlikely(!inode))
|
||||
return eventfs_failed_creating(dentry);
|
||||
|
||||
inode->i_mode = S_IFDIR | S_IRWXU | S_IRUGO | S_IXUGO;
|
||||
update_inode_attr(inode, ef);
|
||||
|
||||
inode->i_op = &eventfs_root_dir_inode_operations;
|
||||
inode->i_fop = &eventfs_file_operations;
|
||||
inode->i_private = data;
|
||||
|
@ -306,10 +380,9 @@ create_dentry(struct eventfs_file *ef, struct dentry *parent, bool lookup)
|
|||
inode_lock(parent->d_inode);
|
||||
|
||||
if (ef->ei)
|
||||
dentry = create_dir(ef->name, parent, ef->data);
|
||||
dentry = create_dir(ef, parent, ef->data);
|
||||
else
|
||||
dentry = create_file(ef->name, ef->mode, parent,
|
||||
ef->data, ef->fop);
|
||||
dentry = create_file(ef, parent, ef->data, ef->fop);
|
||||
|
||||
if (!lookup)
|
||||
inode_unlock(parent->d_inode);
|
||||
|
@ -475,6 +548,7 @@ static int dcache_dir_open_wrapper(struct inode *inode, struct file *file)
|
|||
if (d) {
|
||||
struct dentry **tmp;
|
||||
|
||||
|
||||
tmp = krealloc(dentries, sizeof(d) * (cnt + 2), GFP_KERNEL);
|
||||
if (!tmp)
|
||||
break;
|
||||
|
@ -549,13 +623,14 @@ static struct eventfs_file *eventfs_prepare_ef(const char *name, umode_t mode,
|
|||
return ERR_PTR(-ENOMEM);
|
||||
}
|
||||
INIT_LIST_HEAD(&ef->ei->e_top_files);
|
||||
ef->mode = S_IFDIR | S_IRWXU | S_IRUGO | S_IXUGO;
|
||||
} else {
|
||||
ef->ei = NULL;
|
||||
ef->mode = mode;
|
||||
}
|
||||
|
||||
ef->iop = iop;
|
||||
ef->fop = fop;
|
||||
ef->mode = mode;
|
||||
ef->data = data;
|
||||
return ef;
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue