Fixed - IllegalReferenceCountException is thrown when canceling a method call. #5519

redisson/src/main/java/org/redisson/command/RedisExecutor.java

@@ -132,20 +132,20 @@ public class RedisExecutor<V, R> {
                     return;
                 }
 
-                if (connectionFuture.cancel(false)) {
+                if (connectionFuture.completeExceptionally(new CancellationException())) {
                     log.debug("Connection obtaining canceled for {}", command);
                     timeout.ifPresent(Timeout::cancel);
-                    if (attemptPromise.cancel(false)) {
+                    if (attemptPromise.completeExceptionally(new CancellationException())) {
                         free();
                     }
                 } else {
                     if (command.isBlockingCommand()) {
                         RedisConnection c = connectionFuture.getNow(null);
                         if (writeFuture.cancel(false)) {
-                            attemptPromise.cancel(false);
+                            attemptPromise.completeExceptionally(new CancellationException());
                         } else {
                             c.forceFastReconnectAsync().whenComplete((res, ex) -> {
-                                attemptPromise.cancel(true);
+                                attemptPromise.completeExceptionally(new CancellationException());
                             });
                         }
                     }
@@ -212,7 +212,7 @@ public class RedisExecutor<V, R> {
         timeout.ifPresent(Timeout::cancel);
 
         TimerTask task = timeout -> {
-            if (connectionFuture.cancel(false)) {
+            if (connectionFuture.completeExceptionally(new CancellationException())) {
                 exception = new RedisTimeoutException("Unable to acquire connection! " + this.connectionFuture +
                         "Increase connection pool size or timeout. "
                         + "Node source: " + source
@@ -261,7 +261,7 @@ public class RedisExecutor<V, R> {
                     return;
                 }
 
-                if (connectionFuture.cancel(false)) {
+                if (connectionFuture.completeExceptionally(new CancellationException())) {
                     exception = new RedisTimeoutException("Unable to acquire connection! " + connectionFuture +
                                 "Increase connection pool size. "
                                 + "Node source: " + source
@@ -297,7 +297,7 @@ public class RedisExecutor<V, R> {
                 }
 
                 if (mainPromise.isCancelled()) {
-                    if (attemptPromise.cancel(false)) {
+                    if (attemptPromise.completeExceptionally(new CancellationException())) {
                         free();
                     }
                     return;
@@ -310,7 +310,7 @@ public class RedisExecutor<V, R> {
                     }
                     return;
                 }
-                if (!attemptPromise.cancel(false)) {
+                if (!attemptPromise.completeExceptionally(new CancellationException())) {
                     return;
                 }
 
@@ -407,7 +407,7 @@ public class RedisExecutor<V, R> {
         long timeoutAmount = timeoutTime;
         TimerTask timeoutResponseTask = timeout -> {
             if (isResendAllowed(attempt, attempts)) {
-                if (!attemptPromise.cancel(false)) {
+                if (!attemptPromise.completeExceptionally(new CancellationException())) {
                     return;
                 }
 
@@ -469,7 +469,7 @@ public class RedisExecutor<V, R> {
                         && !attemptPromise.isDone()) {
                 log.debug("Canceled blocking operation {} used {}", command, connection);
                 connection.forceFastReconnectAsync().whenComplete((r, ex) -> {
-                    attemptPromise.cancel(true);
+                    attemptPromise.completeExceptionally(new CancellationException());
                 });
                 return;
             }