From 530678eadf8df0e0720e949793ab34a986ed85e0 Mon Sep 17 00:00:00 2001 From: David McKnight Date: Tue, 12 Aug 2008 16:26:00 +0000 Subject: [PATCH] [243382] [dstore] Server launcher settings are shared by multiple connections --- .../core/subsystems/AbstractConnectorServiceManager.java | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/rse/plugins/org.eclipse.rse.core/src/org/eclipse/rse/core/subsystems/AbstractConnectorServiceManager.java b/rse/plugins/org.eclipse.rse.core/src/org/eclipse/rse/core/subsystems/AbstractConnectorServiceManager.java index 39b917981fe..9319ad49961 100644 --- a/rse/plugins/org.eclipse.rse.core/src/org/eclipse/rse/core/subsystems/AbstractConnectorServiceManager.java +++ b/rse/plugins/org.eclipse.rse.core/src/org/eclipse/rse/core/subsystems/AbstractConnectorServiceManager.java @@ -14,6 +14,7 @@ * Contributors: * Martin Oberhuber (Wind River) - [177523] Unify singleton getter methods * David McKnight (IBM) - [198802] Incorrect logic for getting dummy host + * David McKnight (IBM) - [243382] [dstore] Server launcher settings are shared by multiple connections *******************************************************************************/ package org.eclipse.rse.core.subsystems; @@ -128,7 +129,11 @@ public abstract class AbstractConnectorServiceManager implements IConnectorServi if (key instanceof DummyHost) { IHost host = (IHost)key; - if (host.getHostName().equals(newHost.getHostName())) + // A previous host of the same hostName should not be used since it causes bug 243382. + // A remaining problem here is that we shouldn't keep a host around in + // systemConnectionRegistry after it's corresponding connection has been created + // but at the moment there's no API to do that outside of this class + if (host.equals(newHost)) { Hashtable table = (Hashtable)systemConnectionRegistry.remove(host); systemConnectionRegistry.put(newHost, table);