[#1924] Correctly fail promise when EventExecutor was shutdown in between deregister

2013-10-15 14:01:59 +02:00 · 2013-10-15 14:01:59 +02:00 · 068c75a025
commit 068c75a025
parent 812b0d11c2
1 changed files with 1 additions and 1 deletions
--- a/transport/src/main/java/io/netty/channel/AbstractChannel.java
+++ b/transport/src/main/java/io/netty/channel/AbstractChannel.java
@ -587,13 +587,13 @@ public abstract class AbstractChannel extends DefaultAttributeMap implements Cha
            } finally {
                if (registered) {
                    registered = false;
-                    promise.setSuccess();
                    invokeLater(new Runnable() {
                        @Override
                        public void run() {
                            pipeline.fireChannelUnregistered();
                        }
                    });
+                    promise.setSuccess();
                } else {
                    // Some transports like local and AIO does not allow the deregistration of
                    // an open channel.  Their doDeregister() calls close().  Consequently,