[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

[Rob McKenna]

Approved. Please add appropriate noreg labels to each bug.

	-Rob

On 15/06/16 02:56, Artem Kosarev wrote:
> 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.