[8u-dev] Request for approval for 8027575: b113 causing a lot of memory allocation and regression for wls_webapp_atomics and 8049312: AES/CICO test failed with on several modes

Artem Kosarev artem.kosarev at oracle.com
Wed Jun 15 11:56:42 UTC 2016


Hello,

Please approve almost direct backports of bug fix + regression fix 
(caused by first bug fix) to 8u-dev.

Bug fixes were originally reviewed by Xuelei Fan and Anthony Scarpino.

"b113 causing a lot of memory allocation and regression for 
wls_webapp_atomics"
https://bugs.openjdk.java.net/browse/JDK-8027575
Review thread:
http://mail.openjdk.java.net/pipermail/security-dev/2014-June/010704.html
JDK 9 changeset:
http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/c2e818a8d678

The only changes from original fix is that Copyright year change in one 
file *src/share/classes/com/sun/crypto/provider/CipherCore.java* 
couldn't be applied by mercurial automatically. Needed to do it manually.
*WebRev: **
***http://cr.openjdk.java.net/~akosarev/8027575/webrev.00

Plus:

"AES/CICO test failed with on several modes"
https://bugs.openjdk.java.net/browse/JDK-8049312
Review thread:
http://mail.openjdk.java.net/pipermail/security-dev/2014-December/011519.html
JDK 9 changeset:
http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/09bcd51adf34

The only changes from original fix are the following:
1) I had to remove *java.base* from the pathes of files inside original 
patch from JDK 9. After that mercurial could apply the patch.
2) We don't require to remove test from *ProblemList.txt* file. Test 
hasn't been added to the ProblemList in JDK 8 yet.
*WebRev: **
*http://cr.openjdk.java.net/~akosarev/8049312/webrev.00

Fixes were checked with JPRT.

Thank you,
Artem Kosarev.


More information about the jdk8u-dev mailing list