Hi Jini, 'null' seems to be a valid value for 'data' field in both of the places you are using 'data.clone()' - you should guard for null and call 'clone()' only if the passed in value is non-null.
Cheers, -JB- On Fri, Jul 29, 2016 at 11:29 AM, Jini Susan George <jini.geo...@oracle.com> wrote: > Hi all, > > Please review the fix for the following SA defect (to avoid exposing > internal representations by storing or returning externally mutable > objects directly). > > Bug ID: *https://bugs.openjdk.java.net/browse/JDK-8068004* > <https://bugs.openjdk.java.net/browse/JDK-8068004> > > Webrev: > *http://cr.openjdk.java.net/~sballal/sponsorship/8068004/webrev.00/* > <http://cr.openjdk.java.net/~sballal/sponsorship/8068004/webrev.00/> > > Thanks, > > - Jini Susan George > >
