8254973: CompletableFuture.ThreadPerTaskExecutor does not throw NPE i…

…n #execute Reviewed-by: dl
openjdk · Jan 10, 2021 · 9154f64 · 9154f64 · openjdk-notifier · Jan 10, 2021
1 parent e7c1740
commit 9154f64
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/src/java.base/share/classes/java/util/concurrent/CompletableFuture.java b/src/java.base/share/classes/java/util/concurrent/CompletableFuture.java
@@ -43,6 +43,7 @@
 import java.util.function.Consumer;
 import java.util.function.Function;
 import java.util.function.Supplier;
+import java.util.Objects;
 
 /**
  * A {@link Future} that may be explicitly completed (setting its
@@ -438,7 +439,10 @@ public static interface AsynchronousCompletionTask {
 
     /** Fallback if ForkJoinPool.commonPool() cannot support parallelism */
     static final class ThreadPerTaskExecutor implements Executor {
-        public void execute(Runnable r) { new Thread(r).start(); }
+        public void execute(Runnable r) {
+            Objects.requireNonNull(r);
+            new Thread(r).start();
+        }
     }
 
     /**