Integrated: 8287073: NPE from CgroupV2Subsystem.getInstance()

Mon May 30 06:22:33 UTC 2022

On Fri, 20 May 2022 08:34:58 GMT, Maxim Kartashev <mkartashev at 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 <mkartashev at openjdk.org>
Committer: Ioi Lam <iklam at 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