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