RFR: 8322420: [Linux] cgroup v2: Limits in parent nested control groups are not detected [v2]
Severin Gehwolf
sgehwolf at openjdk.org
Wed Aug 28 09:00:34 UTC 2024
On Wed, 28 Aug 2024 08:46:35 GMT, Severin Gehwolf <sgehwolf at openjdk.org> wrote:
>> You free _path here, but it is still not nullptr. You then rely on all three code paths below to reset _path to a sensible value. That is brittle and may bitrot. It would be cleaner to add a _path = nullptr right after the free, to prevent the old invalif pointer from being accidentally used.
>
> Ah, makes sense.
Done now.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20646#discussion_r1734282181
More information about the hotspot-runtime-dev
mailing list