Looks good to me.
--Sean
On 06/06/2014 06:16 PM, Jamil Nimeh wrote:
One more version of this webrev with minor comment changes:
http://cr.openjdk.java.net/~ascarpino/8015081/webrev.04
Thanks,
--Jamil
On 06/04/2014 04:29 PM, Jamil Nimeh wrote:
Hello all,
This is an update to the webrev for JDK-8015081 that takes into
account review changes and adds a few more tests.
http://cr.openjdk.java.net/~ascarpino/8015081/webrev.03
<http://cr.openjdk.java.net/%7Eascarpino/8015081/webrev.03>
Thanks!
--Jamil
On 05/27/2014 05:53 PM, Jamil Nimeh wrote:
Hello all,
This bug was originally to resolve issues where Subject principal and
credential Set fields containing null elements could cause
NullPointerException to be thrown. It was decided to make the
Subject throw NullPointerException when attempts to construct or
modify Subjects will null elements. In addition, certain Set methods
called upon principals and credentials where the input Collection
contained null elements could have different effects depending on
which set. These methods, when called with Collections containing
null elements as input now throw NullPointerException as well.
Operation Before After
(Prin/Pub/Priv) (Prin/Pub/Priv) (Prin/Pub/Priv)
----------------------------------------------------------------------------
add(null) SecurityException/true/true NPE/NPE/NPE
remove(null) false/false/false (*) NPE/NPE/NPE
contains(null) false/false/false (*) NPE/NPE/NPE
addAll(collection w/ null) SecurityException/true/true NPE/NPE/NPE
removeAll(collection w/ null) true/true/true (**) NPE/NPE/NPE
containsAll(coll. w/ null) false/false/NPE NPE/NPE/NPE
retainAll(collection w/ null) true/true/true (***) NPE/NPE/NPE
* - Assumes Subject has no null values in its private SecureSet
collections
** - Assumes input collection has at least one non-null element that
exists in the target SecureSet
*** - Assumes input collection is not equivalent to or a superset of
the target SecureSet (which would cause no change)
Bug: https://bugs.openjdk.java.net/browse/JDK-8015081
Webrev: http://cr.openjdk.java.net/~ascarpino/8015081/webrev.02
Thanks,
--Jamil
