Re: [PATCH v2 1/3] exec: Dynamically allocate memory to store task's full name

Kees Cook
2025-04-04T17:24:41Z

On Fri, Apr 04, 2025 at 12:18:56PM +0530, Bhupesh Sharma wrote:
> In another review for this series, Yafang mentioned the following cleanup +
> approach suggested by Linus (see [0]).
> Also I have summarized my understanding on the basis of the suggestions
> Linus shared and the accompanying background threads (please see [1]).
>
> Kindly share your views on the same, so that I can change the implementation
> in v3 series accordingly.

In thinking about this a little more I think we can't universally change
all the APIs to use the new full_name since it is a pointer, which may
be getting changed out from under readers if a setter changes it. So
this may need some careful redesign, likely with RCU. hmm.

--
Kees Cook