diff options
author | Aleksa Sarai <cyphar@cyphar.com> | 2019-12-06 15:13:27 +0100 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2019-12-09 01:09:37 +0100 |
commit | ce623f89872df4253719be71531116751eeab85f (patch) | |
tree | a47f85060eeaa4e4d0a277f81dab29d824fd0c33 /include | |
parent | namei: only return -ECHILD from follow_dotdot_rcu() (diff) | |
download | linux-ce623f89872df4253719be71531116751eeab85f.tar.xz linux-ce623f89872df4253719be71531116751eeab85f.zip |
nsfs: clean-up ns_get_path() signature to return int
ns_get_path() and ns_get_path_cb() only ever return either NULL or an
ERR_PTR. It is far more idiomatic to simply return an integer, and it
makes all of the callers of ns_get_path() more straightforward to read.
Fixes: e149ed2b805f ("take the targets of /proc/*/ns/* symlinks to separate fs")
Signed-off-by: Aleksa Sarai <cyphar@cyphar.com>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'include')
-rw-r--r-- | include/linux/proc_ns.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/linux/proc_ns.h b/include/linux/proc_ns.h index d31cb6215905..aed366b4795c 100644 --- a/include/linux/proc_ns.h +++ b/include/linux/proc_ns.h @@ -76,10 +76,10 @@ static inline int ns_alloc_inum(struct ns_common *ns) extern struct file *proc_ns_fget(int fd); #define get_proc_ns(inode) ((struct ns_common *)(inode)->i_private) -extern void *ns_get_path(struct path *path, struct task_struct *task, +extern int ns_get_path(struct path *path, struct task_struct *task, const struct proc_ns_operations *ns_ops); typedef struct ns_common *ns_get_path_helper_t(void *); -extern void *ns_get_path_cb(struct path *path, ns_get_path_helper_t ns_get_cb, +extern int ns_get_path_cb(struct path *path, ns_get_path_helper_t ns_get_cb, void *private_data); extern int ns_get_name(char *buf, size_t size, struct task_struct *task, |