Re: [PATCH 12/17] proc/fd: In fdinfo seq_show don't use get_files_struct
From: Linus Torvalds
Date: Mon Aug 17 2020 - 21:28:38 EST
On Mon, Aug 17, 2020 at 6:13 PM Eric W. Biederman <ebiederm@xxxxxxxxxxxx> wrote:
>
> task_lock is needed to protect task->files.
Hah. Right you are. I found a few cases where we didn't do that, but I
hadn't noticed that they were all of the pattern
struct task_struct *tsk = current;
so "tsk->files" was safe for that reason..
So never mind.
Linus