On Fri, 20 May 2022 08:34:58 GMT, Maxim Kartashev <mkartas...@openjdk.org> wrote:
> Following the logic from the comment directly above the changed line, since > it doesn't matter which controller we pick, pick any available controller > instead of the one called "memory" specifically. This way we are guarded > against getting `null` as `anyController`, which is being immediately passed > down to `CgroupV2Subsystem.getInstance()` that is unprepared to accept `null` > values. > > It is also worth noting that the previous checks (such as that at line 89) > make sure that there exist at least one controller in the map. This pull request has now been integrated. Changeset: 744b822a Author: Maxim Kartashev <mkartas...@openjdk.org> Committer: Ioi Lam <ik...@openjdk.org> URL: https://git.openjdk.java.net/jdk/commit/744b822ab194a0f7ef4e7a4053be32c6a0889efc Stats: 4 lines in 1 file changed: 1 ins; 0 del; 3 mod 8287073: NPE from CgroupV2Subsystem.getInstance() Reviewed-by: vkempik, iklam ------------- PR: https://git.openjdk.java.net/jdk/pull/8803
