RFR: 8025067: (xs) Arrays.parallelPrefix may fail to throw NPE for null op

Mike Duigou mike.duigou at oracle.com
Mon Sep 30 22:39:07 UTC 2013


After posting the webrev I decided that an update to the regression test was going to be necessary. Here's the updated webrev including additional unit tests. Unfortunately I had to cut-and-paste some "library" code for exception checking. The "library" code will need to be cleaned up eventually.

http://cr.openjdk.java.net/~mduigou/JDK-8025067/1/webrev/

Mike

On Sep 30 2013, at 15:00 , Henry Jen wrote:

> Looks good to me.
> 
> Cheers,
> Henry
> 
> On Sep 30, 2013, at 2:27 PM, Mike Duigou <mike.duigou at oracle.com> wrote:
> 
>> Hello all;
>> 
>> This changeset fixes a small corner case where Arrays.parallelPrefix (all variants) failed to throw NPE if the operation was null. If the array or range was empty then op was never examined. The solution in this patch is to add an an explicit Objects.requireNonNull(op) check.
>> 
>> http://cr.openjdk.java.net/~mduigou/JDK-8025067/0/webrev/
>> 
>> Cheers,
>> 
>> Mike
> 




More information about the core-libs-dev mailing list