mirrors/cdt
1
0
Fork 0
mirror of https://github.com/eclipse-cdt/cdt synced 2025-07-03 15:15:25 +02:00
Code Issues Wiki Activity

[235577] [dstore][launcher] Number 0 is accepted as valid port number

Browse source
This commit is contained in:
David McKnight 2008-07-07 15:57:38 +00:00
parent 9afbd5020f
commit 9bbc6bb137
1 changed files with 14 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
rse/plugins/org.eclipse.rse.ui/UI/org/eclipse/rse/ui/widgets/RemoteServerLauncherForm.java
View file

@ -13,6 +13,7 @@
*
* Contributors:
* Martin Oberhuber (Wind River) - [226364][api][breaking] RemoteBaseServerLauncherForm should not implement RemoteServerLauncherConstants.
* David McKnight (IBM) - [235577] [dstore][launcher] Number 0 is accepted as valid port number
*******************************************************************************/
package org.eclipse.rse.ui.widgets;
@ -302,8 +303,14 @@ public class RemoteServerLauncherForm extends RemoteBaseServerLauncherForm
if (launchType == ServerLaunchType.REXEC_LITERAL)
{
String port = getREXECPort();
int portAsInt = Integer.parseInt(port);
msg = _rexecPortValidator.validate(port);
// for daemons we don't allow 0
if (msg == null && portAsInt == 0)
msg = RSEUIPlugin.getPluginMessage(ISystemMessages.MSG_VALIDATE_PORT_NOTVALID);
if (msg == null)
{
String path = getServerInstallPath();
@ -330,8 +337,15 @@ public class RemoteServerLauncherForm extends RemoteBaseServerLauncherForm
else if (launchType == ServerLaunchType.DAEMON_LITERAL)
{
String port = getDaemonPort();
int portAsInt = Integer.parseInt(port);
msg = _daemonPortValidator.validate(port);
// for daemons we don't allow 0
if (msg == null && portAsInt == 0)
msg = RSEUIPlugin.getPluginMessage(ISystemMessages.MSG_VALIDATE_PORT_NOTVALID);
if (msg == null)
{
_msgLine.clearErrorMessage();