> On 19 Sep 2019, at 18:32, Joe Darcy <joe.da...@oracle.com> wrote:
> 
> Hello,
> 
> Ahead of augmenting javac's serial lint checks under JDK-8160675, it would be 
> helpful to mark fields in security libs classes where the class is 
> serializable, but a non-transient instance field does *not* have a 
> serialiable type. Such classes may have difficulties being serialized at 
> runtime:
> 
>     JDK-8231262 : Suppress warnings on non-serializable instance fields in 
> security libs serializable classes
>     http://cr.openjdk.java.net/~darcy/8231262.0/

The changes look good to me.

The fields in PrivateCredentialPermission and SecureRandom, could be made final 
and assigned null, ensuring non-Serializable types will never leak into them. 
But equally, this could be left to a follow on change for someone working in 
the security area.

-Chris.

Reply via email to