diff options
| author | Mateusz Guzik <mguzik@redhat.com> | 2016-08-23 16:20:38 +0200 |
|---|---|---|
| committer | Paul Moore <paul@paul-moore.com> | 2016-08-31 22:11:20 +0200 |
| commit | cd81a9170e69e018bbaba547c1fd85a585f5697a (patch) | |
| tree | b0c11dc4cc55aba04e21f8c7352b8c516124cbdb /fs/proc | |
| parent | audit: fix a double fetch in audit_log_single_execve_arg() (diff) | |
| download | linux-cd81a9170e69e018bbaba547c1fd85a585f5697a.tar.xz linux-cd81a9170e69e018bbaba547c1fd85a585f5697a.zip | |
mm: introduce get_task_exe_file
For more convenient access if one has a pointer to the task.
As a minor nit take advantage of the fact that only task lock + rcu are
needed to safely grab ->exe_file. This saves mm refcount dance.
Use the helper in proc_exe_link.
Signed-off-by: Mateusz Guzik <mguzik@redhat.com>
Acked-by: Konstantin Khlebnikov <khlebnikov@yandex-team.ru>
Acked-by: Richard Guy Briggs <rgb@redhat.com>
Cc: <stable@vger.kernel.org> # 4.3.x
Signed-off-by: Paul Moore <paul@paul-moore.com>
Diffstat (limited to 'fs/proc')
| -rw-r--r-- | fs/proc/base.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/fs/proc/base.c b/fs/proc/base.c index 0d163a84082d..da8b1943ba04 100644 --- a/fs/proc/base.c +++ b/fs/proc/base.c @@ -1552,18 +1552,13 @@ static const struct file_operations proc_pid_set_comm_operations = { static int proc_exe_link(struct dentry *dentry, struct path *exe_path) { struct task_struct *task; - struct mm_struct *mm; struct file *exe_file; task = get_proc_task(d_inode(dentry)); if (!task) return -ENOENT; - mm = get_task_mm(task); + exe_file = get_task_exe_file(task); put_task_struct(task); - if (!mm) - return -ENOENT; - exe_file = get_mm_exe_file(mm); - mmput(mm); if (exe_file) { *exe_path = exe_file->f_path; path_get(&exe_file->f_path); |