idleConnectionTimeout might have been dropped or not set, close AsyncHttpClient#534

Author: Stephane Landelle

providers/netty/src/main/java/org/asynchttpclient/providers/netty/request/timeout/TimeoutsHolder.java

@@ -21,16 +21,20 @@
 
 public class TimeoutsHolder {
 
-    private AtomicBoolean cancelled = new AtomicBoolean();
+    private final AtomicBoolean cancelled = new AtomicBoolean();
     public volatile Timeout requestTimeout;
     public volatile Timeout idleConnectionTimeout;
 
     public void cancel() {
         if (cancelled.compareAndSet(false, true)) {
-            requestTimeout.cancel();
-            idleConnectionTimeout.cancel();
-            requestTimeout = null;
-            idleConnectionTimeout = null;
+            if (requestTimeout != null) {
+                requestTimeout.cancel();
+                requestTimeout = null;
+            }
+            if (idleConnectionTimeout != null) {
+                idleConnectionTimeout.cancel();
+                idleConnectionTimeout = null;
+            }
         }
     }
 }