[jboss-jira] [JBoss JIRA] Commented: (JGRP-580) Possible to get the channel into an incorrect and/or inconsistent state

Possible to get the channel into an incorrect and/or inconsistent state ----------------------------------------------------------------------- Key: JGRP-580 URL: http://jira.jboss.com/jira/browse/JGRP-580 Project: JGroups Issue Type: Bug Affects Versions: 2.5 Environment: Windows XP; java 1.5.0_12 Reporter: Anders Persson Assigned To: Bela Ban Fix For: 2.6 The channel is trying to establish a connection. While trying to create the connection there is a Event.EXIT traversed up the stack. This will close the channel and at the same time release connect(). The problem is that when the thread returns to connect() it will continue as if the connect is continuing successfully. This will result in a channel which is set to be connected even though it is actually closed. This will also cause the auto reconnect to fail since the channel considers itself to be connected. After this long description I think that the following snippet of code will provide a better description. Today in JChannel the code looks like this: public synchronized void connect(String cluster_name) throws ChannelException { ...stuff going on... Event connect_event = new Event(Event.CONNECT, cluster_name); Object res = downcall(connect_event); // waits forever until // connected (or channel is // closed) if (res != null && res instanceof Exception) { // the JOIN was // rejected by the // coordinator throw new ChannelException("connect() failed", (Throwable) res); } This should be changed to: Event connect_event = new Event(Event.CONNECT, cluster_name); Object res = downcall(connect_event); // waits forever until // connected (or channel is // closed) /* * This is an added section to this method. The if statement will * check if the downcall method returned as a result of an * Event.EXIT. If so then we should consider this channel to be NOT * connected. Otherwise connected will be set to true and any * further calls to connect will fail (also auto reconnect will fail * for this reason). */ if (!closed) { if (res != null && res instanceof Exception) { // the JOIN was // rejected by the // coordinator throw new ChannelException("connect() failed", (Throwable) res); } It obviously has to be verified that the closed flag is set before calling any stop methods on the protocols.