JDK-8196374 : windows x86 webview-icu isAlphaNumericString crash
Type:Bug
Component:javafx
Sub-Component:web
Affected Version:8u161
Priority:P2
Status:Resolved
Resolution:Fixed
OS:windows
CPU:x86
Submitted:2018-01-30
Updated:2018-06-13
Resolved:2018-01-30
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.
Load http://www.wikipedia.org using webview
or load simplified html page with "<a href='' lang='roa-x-eml'>��</a> </body>"
Actual : Application crash (hs_err_pid12760.log)
Expected : Render page
Comments
CPU18_04-critical-request
- Justification : This bug fixes crash in icu library due to Visual studio code optimisation on x86 (32 bit) build
- Risk Analysis : Low. Disables code optimisation for only block of source (only for ultag_parse function)
- Review : http://cr.openjdk.java.net/~ghb/8196374/webrev.00/
- Testing (done/to-be-done) : done, provided unit test
- Back ports (done/to-be-done) : done to 10u-cpu-dev(10.0.2), 8u-dev(8u182)
- Fix For Release : 8u171, 10.0.1
Note that this fix should still be pushed to jfx-dev (for 11) both to keep the code in sync, and in case someone wants to produce 32-bit binaries for jfx-dev.
30-01-2018
+1
30-01-2018
+1
30-01-2018
lgtm
30-01-2018
Webrev : http://cr.openjdk.java.net/~ghb/8196374/webrev.00/
Cherry pick the solution from "http://bugs.icu-project.org/trac/changeset/39286"
Root cause and its work around updated in http://bugs.icu-project.org/trac/ticket/12705
Tested with simple HelloWebView with loading http://www.wikipedia.org or create a simple html file with content "<a href="" lang="roa-x-eml">��</a>".
however WebView.loadContent("<a href="" lang="roa-x-eml">��</a>"); will not crash, where as loading from an html file containing this simplified 'lang' identifier 'roa-x-eml' along with a content '��'.
I observed this problem only on 32bit (client) JDK, tried testing JFXTest using 8u161b12 x64 didn't crash on my PC (with or without my fix).
roa-x-eml : lang identifier of Emilian egl and Romagnoal rgn. Didn't get much info about why only with this unknown identifier (not specified in icu lang list) crashes.