Re: RFR : 8159035: com/sun/crypto/provider/Cipher/CTS/CTSMode.java test crashed due to unhandled case of cipher length value as 0

2017-08-25 Thread Anthony Scarpino
Looks good Tony > On Aug 25, 2017, at 6:28 AM, Seán Coffey wrote: > > Tony pointed out off-list that the change to encodeISOArray() might not work. > > "I'm not certain the change to encodeISOArray() is not going to work. In 8, > that method can be replaced by an intrinsic and miss your addit

Re: RFR : 8159035: com/sun/crypto/provider/Cipher/CTS/CTSMode.java test crashed due to unhandled case of cipher length value as 0

2017-08-25 Thread Seán Coffey
Tony pointed out off-list that the change to encodeISOArray() might not work. "I'm not certain the change to encodeISOArray() is not going to work. In 8, that method can be replaced by an intrinsic and miss your additional check. Unlike in 9 where implEncodeISOArray() is intrinsfied. It mayb

RFR : 8159035: com/sun/crypto/provider/Cipher/CTS/CTSMode.java test crashed due to unhandled case of cipher length value as 0

2017-08-16 Thread Seán Coffey
Looking to backport this fix to jdk8u-dev. The bug report is not public but the JDK 9 review thread captures the details. The fix is similar. Some extra intrinsics work in jdk9 wasn't applicable for the jdk8u-dev port. http://mail.openjdk.java.net/pipermail/core-libs-dev/2016-November/044560.