JDK-8232926 : NPE regression after 8216064
  • Type: Bug
  • Component: hotspot
  • Sub-Component: jfr
  • Affected Version: openjdk8u-jfr-incubator
  • Priority: P3
  • Status: New
  • Resolution: Unresolved
  • OS: generic
  • CPU: generic
  • Submitted: 2019-10-24
  • Updated: 2024-02-10
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
Other
openjdk8uUnresolved
Related Reports
Relates :  
JDK-8216064 - -XX:StartFlightRecording:settings= doesn't work properly
Description
The 8u backport of 8216064 introduces a regression when "settings" is not specified. In particular, this line here [1] may trigger a NPE when "settings" is null. Please notice how that case is handled here [2].

Update: seems to me that JDK baseline might be affected by this issue too.

--
[1] - http://hg.openjdk.java.net/jdk8u/jdk8u-jfr-incubator/jdk/rev/aaf089a18674#l1.17
[2] - http://hg.openjdk.java.net/jdk8u/jdk8u-jfr-incubator/jdk/rev/aaf089a18674#l1.22
Comments
How is the NPE triggered? Judging just by the DCMD code that should be the only one executing this method (https://github.com/openjdk/jdk8u-dev/blob/cfed45d15a36cd92aec873f38c46cd085ee0b9b1/hotspot/src/share/vm/jfr/dcmd/jfrDcmds.cpp#L452) we should never end up with `null` value for `settings`.
10-02-2024