proc: Put thread_pid in release_task not proc_flush_pid
authorEric W. Biederman <ebiederm@xmission.com>
Fri, 24 Apr 2020 20:41:20 +0000 (15:41 -0500)
committerEric W. Biederman <ebiederm@xmission.com>
Fri, 24 Apr 2020 20:49:00 +0000 (15:49 -0500)
commit6ade99ec6175ab2b54c227521e181e1c3c2bfc8a
treebcbb3ed8d7cb60920556673bd571a8206c21e759
parent61e713bdca3678e84815f2427f7a063fc353a1fc
proc: Put thread_pid in release_task not proc_flush_pid

Oleg pointed out that in the unlikely event the kernel is compiled
with CONFIG_PROC_FS unset that release_task will now leak the pid.

Move the put_pid out of proc_flush_pid into release_task to fix this
and to guarantee I don't make that mistake again.

When possible it makes sense to keep get and put in the same function
so it can easily been seen how they pair up.

Fixes: 7bc3e6e55acf ("proc: Use a list of inodes to flush from proc")
Reported-by: Oleg Nesterov <oleg@redhat.com>
Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com>
fs/proc/base.c
kernel/exit.c