Re: [PATCH v2 1/3] exec: Dynamically allocate memory to store task's full name
From: Kees Cook
Date: Fri Apr 04 2025 - 13:24:52 EST
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