On Tue, Feb 15, 2022 at 12:05 PM Michal Hocko <mhocko@xxxxxxxx> wrote: > > One thing I was considering is to check agains ref counte overflo (a > deep process chain with many vmas could grow really high. ref_count > interface doesn't provide any easy way to check for overflows as far as > I could see from a quick glance so I gave up there but the logic would > be really straightforward. We just create a new anon_vma_name with the same > content and use it when duplicating if the usage grow really > (arbitrarily) high. I went over proposed changes. I see a couple small required fixes (resetting the name to NULL seems to be missing and I think dup_vma_anon_name needs some tweaking) but overall quite straight-forward. I'll post a separate patch to do this refactoring. The original patch is fixing the UAF issue, so I don't want to mix it with refactoring. Please let me know if you see an issue with separating it that way. Thanks, Suren. > -- > Michal Hocko > SUSE Labs
