On Wed, Sep 01, 2021 at 10:00:40PM +0700, Dmitry Kadashev wrote: > After the switch of kern_path_locked() to __filename_parentat() (to > address use after free bug) nothing is using filename_parentat(). Also, > filename_parentat() is inherently buggy: the "last" output arg > always point to freed memory. > > Drop filename_parentat() and rename __filename_parentat() to > filename_parentat(). I'd rather fold that into previous patch. And it might be better to fold filename_create() into its 2 callers and rename __filename_create() as well. Let me poke around a bit...