On 29.01.2015 0:24, Michael StJohns wrote:
Not critical and more work, but maybe just update this to use java.util.BitSet?
I'd prefer not to invest much effort in refactoring of this class.
Let's limit scope of the fix by only getting rid of AIOOB exception for
positive indices!
Sincerely yours,
Ivan
Mike
At 04:04 PM 1/28/2015, Ivan Gerasimov wrote:
Hello!
Behavior of sun.security.x509.NetscapeCertTypeExtension turns out to be
inconsistent.
It greatly depends on the order in which the entries are set/retrieved.
The inconsistency is illustrated in the regression test:
While these two lines work fine,
38 ext1.set(SSL_SERVER, true);
39 ext1.get(SSL_CLIENT);
next two lines result in AIOOBEx:
43 ext2.set(SSL_CLIENT, true);
44 ext2.get(SSL_SERVER);
The fix is trivial.
Would you please help review it?
BUGURL: https://bugs.openjdk.java.net/browse/JDK-8068720
WEBREV: http://cr.openjdk.java.net/~igerasim/8068720/0/webrev/
Sincerely yours,
Ivan