8254350: CompletableFuture.get may swallow InterruptedException
Reviewed-by: alanb, dl
This commit is contained in:
parent
6d79ec87fb
commit
43dc3f7992
src/java.base/share/classes/java/util/concurrent
test/jdk/java/util/concurrent/CompletableFuture
@ -1871,6 +1871,8 @@ public class CompletableFuture<T> implements Future<T>, CompletionStage<T> {
|
|||||||
* interrupted.
|
* interrupted.
|
||||||
*/
|
*/
|
||||||
private Object waitingGet(boolean interruptible) {
|
private Object waitingGet(boolean interruptible) {
|
||||||
|
if (interruptible && Thread.interrupted())
|
||||||
|
return null;
|
||||||
Signaller q = null;
|
Signaller q = null;
|
||||||
boolean queued = false;
|
boolean queued = false;
|
||||||
Object r;
|
Object r;
|
||||||
@ -1882,25 +1884,25 @@ public class CompletableFuture<T> implements Future<T>, CompletionStage<T> {
|
|||||||
}
|
}
|
||||||
else if (!queued)
|
else if (!queued)
|
||||||
queued = tryPushStack(q);
|
queued = tryPushStack(q);
|
||||||
|
else if (interruptible && q.interrupted) {
|
||||||
|
q.thread = null;
|
||||||
|
cleanStack();
|
||||||
|
return null;
|
||||||
|
}
|
||||||
else {
|
else {
|
||||||
try {
|
try {
|
||||||
ForkJoinPool.managedBlock(q);
|
ForkJoinPool.managedBlock(q);
|
||||||
} catch (InterruptedException ie) { // currently cannot happen
|
} catch (InterruptedException ie) { // currently cannot happen
|
||||||
q.interrupted = true;
|
q.interrupted = true;
|
||||||
}
|
}
|
||||||
if (q.interrupted && interruptible)
|
|
||||||
break;
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
if (q != null && queued) {
|
if (q != null) {
|
||||||
q.thread = null;
|
q.thread = null;
|
||||||
if (!interruptible && q.interrupted)
|
if (q.interrupted)
|
||||||
Thread.currentThread().interrupt();
|
Thread.currentThread().interrupt();
|
||||||
if (r == null)
|
|
||||||
cleanStack();
|
|
||||||
}
|
}
|
||||||
if (r != null || (r = result) != null)
|
postComplete();
|
||||||
postComplete();
|
|
||||||
return r;
|
return r;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -0,0 +1,73 @@
|
|||||||
|
/*
|
||||||
|
* Copyright (c) 2020, Oracle and/or its affiliates. All rights reserved.
|
||||||
|
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
|
||||||
|
*
|
||||||
|
* This code is free software; you can redistribute it and/or modify it
|
||||||
|
* under the terms of the GNU General Public License version 2 only, as
|
||||||
|
* published by the Free Software Foundation.
|
||||||
|
*
|
||||||
|
* This code is distributed in the hope that it will be useful, but WITHOUT
|
||||||
|
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
|
||||||
|
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
|
||||||
|
* version 2 for more details (a copy is included in the LICENSE file that
|
||||||
|
* accompanied this code).
|
||||||
|
*
|
||||||
|
* You should have received a copy of the GNU General Public License version
|
||||||
|
* 2 along with this work; if not, write to the Free Software Foundation,
|
||||||
|
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
|
||||||
|
*
|
||||||
|
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
|
||||||
|
* or visit www.oracle.com if you need additional information or have any
|
||||||
|
* questions.
|
||||||
|
*/
|
||||||
|
|
||||||
|
import java.util.concurrent.CompletableFuture;
|
||||||
|
import java.util.concurrent.ForkJoinPool;
|
||||||
|
import java.util.concurrent.ThreadLocalRandom;
|
||||||
|
import static java.util.concurrent.TimeUnit.DAYS;
|
||||||
|
|
||||||
|
/*
|
||||||
|
* @test
|
||||||
|
* @bug 8254350
|
||||||
|
* @run main LostInterrupt
|
||||||
|
* @summary CompletableFuture.get may swallow interrupt status
|
||||||
|
* @key randomness
|
||||||
|
*/
|
||||||
|
|
||||||
|
// TODO: Rewrite as a CompletableFuture tck test ?
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Submits a task that completes immediately, then invokes CompletableFuture.get
|
||||||
|
* with the interrupt status set. CompletableFuture.get should either complete
|
||||||
|
* immediately with the interrupt status set, or else throw InterruptedException
|
||||||
|
* with the interrupt status cleared.
|
||||||
|
*/
|
||||||
|
public class LostInterrupt {
|
||||||
|
static final int ITERATIONS = 10_000;
|
||||||
|
|
||||||
|
public static void main(String[] args) throws Exception {
|
||||||
|
ThreadLocalRandom rnd = ThreadLocalRandom.current();
|
||||||
|
ForkJoinPool executor = new ForkJoinPool(1);
|
||||||
|
try {
|
||||||
|
for (int i = 0; i < ITERATIONS; i++) {
|
||||||
|
CompletableFuture<String> future = new CompletableFuture<>();
|
||||||
|
boolean timed = rnd.nextBoolean();
|
||||||
|
executor.execute(() -> future.complete("foo"));
|
||||||
|
|
||||||
|
Thread.currentThread().interrupt();
|
||||||
|
try {
|
||||||
|
String result = timed ? future.get(1, DAYS) : future.get();
|
||||||
|
|
||||||
|
if (!Thread.interrupted())
|
||||||
|
throw new AssertionError("lost interrupt, run=" + i);
|
||||||
|
} catch (InterruptedException expected) {
|
||||||
|
if (Thread.interrupted())
|
||||||
|
throw new AssertionError(
|
||||||
|
"interrupt status not cleared, run=" + i);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
} finally {
|
||||||
|
executor.shutdown();
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
@ -0,0 +1,90 @@
|
|||||||
|
/*
|
||||||
|
* Copyright (c) 2020, Oracle and/or its affiliates. All rights reserved.
|
||||||
|
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
|
||||||
|
*
|
||||||
|
* This code is free software; you can redistribute it and/or modify it
|
||||||
|
* under the terms of the GNU General Public License version 2 only, as
|
||||||
|
* published by the Free Software Foundation.
|
||||||
|
*
|
||||||
|
* This code is distributed in the hope that it will be useful, but WITHOUT
|
||||||
|
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
|
||||||
|
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
|
||||||
|
* version 2 for more details (a copy is included in the LICENSE file that
|
||||||
|
* accompanied this code).
|
||||||
|
*
|
||||||
|
* You should have received a copy of the GNU General Public License version
|
||||||
|
* 2 along with this work; if not, write to the Free Software Foundation,
|
||||||
|
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
|
||||||
|
*
|
||||||
|
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
|
||||||
|
* or visit www.oracle.com if you need additional information or have any
|
||||||
|
* questions.
|
||||||
|
*/
|
||||||
|
|
||||||
|
import java.util.concurrent.CompletableFuture;
|
||||||
|
import java.util.concurrent.CountDownLatch;
|
||||||
|
import java.util.concurrent.ThreadLocalRandom;
|
||||||
|
import java.util.concurrent.atomic.AtomicReference;
|
||||||
|
|
||||||
|
/*
|
||||||
|
* @test
|
||||||
|
* @bug 8254350
|
||||||
|
* @run main SwallowedInterruptedException
|
||||||
|
* @key randomness
|
||||||
|
*/
|
||||||
|
|
||||||
|
public class SwallowedInterruptedException {
|
||||||
|
static final int ITERATIONS = 100;
|
||||||
|
|
||||||
|
public static void main(String[] args) throws Throwable {
|
||||||
|
for (int i = 1; i <= ITERATIONS; i++) {
|
||||||
|
System.out.format("Iteration %d%n", i);
|
||||||
|
|
||||||
|
CompletableFuture<Void> future = new CompletableFuture<>();
|
||||||
|
CountDownLatch running = new CountDownLatch(1);
|
||||||
|
AtomicReference<String> failed = new AtomicReference<>();
|
||||||
|
|
||||||
|
Thread thread = new Thread(() -> {
|
||||||
|
// signal main thread that child is running
|
||||||
|
running.countDown();
|
||||||
|
|
||||||
|
// invoke Future.get, it complete with the interrupt status set or
|
||||||
|
// else throw InterruptedException with the interrupt status not set.
|
||||||
|
try {
|
||||||
|
future.get();
|
||||||
|
|
||||||
|
// interrupt status should be set
|
||||||
|
if (!Thread.currentThread().isInterrupted()) {
|
||||||
|
failed.set("Future.get completed with interrupt status not set");
|
||||||
|
}
|
||||||
|
} catch (InterruptedException ex) {
|
||||||
|
// interrupt status should be cleared
|
||||||
|
if (Thread.currentThread().isInterrupted()) {
|
||||||
|
failed.set("InterruptedException with interrupt status set");
|
||||||
|
}
|
||||||
|
} catch (Throwable ex) {
|
||||||
|
failed.set("Unexpected exception " + ex);
|
||||||
|
}
|
||||||
|
});
|
||||||
|
thread.setDaemon(true);
|
||||||
|
thread.start();
|
||||||
|
|
||||||
|
// wait for thread to run
|
||||||
|
running.await();
|
||||||
|
|
||||||
|
// interrupt thread and set result after an optional (random) delay
|
||||||
|
thread.interrupt();
|
||||||
|
long sleepMillis = ThreadLocalRandom.current().nextLong(10);
|
||||||
|
if (sleepMillis > 0)
|
||||||
|
Thread.sleep(sleepMillis);
|
||||||
|
future.complete(null);
|
||||||
|
|
||||||
|
// wait for thread to terminate and check for failure
|
||||||
|
thread.join();
|
||||||
|
String failedReason = failed.get();
|
||||||
|
if (failedReason != null) {
|
||||||
|
throw new RuntimeException("Test failed: " + failedReason);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
Loading…
x
Reference in New Issue
Block a user