JDK-8069002 : REGRESSION: test/script/external/test262/test/suite/ch11/11.2/11.2.3/S11.2.3_A3_T5.js fails with tip
  • Type: Bug
  • Component: core-libs
  • Sub-Component: jdk.nashorn
  • Priority: P2
  • Status: Resolved
  • Resolution: Fixed
  • OS: generic
  • CPU: generic
  • Submitted: 2015-01-14
  • Updated: 2015-09-29
  • Resolved: 2015-01-14
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.
JDK 8 JDK 9
8u40Fixed 9 b47Fixed
Related Reports
Relates :  
JDK-8068889 - Calling a @FunctionalInterface from JS leaks internal objects
Description
test/script/external/test262/test/suite/ch11/11.2/11.2.3/S11.2.3_A3_T5.js fails

This causes the case of invoking null in JS to throw java.lang.NullPointerException. It is expected to throw a JavaScript specific TypeError instead. Examples of such invocations are

  null();

or

  var x = null;
  ...
  x();
Comments
SQE is ok to take the fix in 8u40.
16-01-2015
Critical Request Template - Justification : this fixes a corner case regression that was introduced in JDK-8068889, which itself is submitted for consideration as a critical fix. If 8068889 is approved, this should be approved as well. - Risk Analysis : No risk. The code prevents a NPE when (in JavaScript) a null value is invoked as a function. The specification prescribes that in such a case a JavaScript TypeError is to be throw instead of the Java specific NPE. The fix is addition of a single null check in appropriate place. - Webrev : http://mail.openjdk.java.net/pipermail/nashorn-dev/2015-January/004101.html - Testing (done/to-be-done) : A new test has been added to Nashorn's test suite (test/script/basic/JDK-8069002.js). The test262 test suite has been run as well and it indicated 0 failures (this issue was originally discovered as a failure in one test262 test). - Back ports (done/to-be-done) : Backported to 8u-dev; 8u-dev backport needs to be pushed into 8u40 if approved here. No further backports necessary. - FX Impact : N/A - Fix For Release : 8u40
16-01-2015
Moving code from NashornBottomLinker to NashornBeansLinker didn't account for the fact NashornBottomLinker also handled null linking separately.
14-01-2015