Inproper parsing of ORBTCPTimeouts

[glassfishrobot]

When setting TCP timeouts per this article:
http://blogs.sun.com/ejcorba/entry/more_on_tcp_timeouts_in

By setting the setting the System property:
-Dcom.sun.corba.ee.transport.ORBTCPTimeouts=500:30000:20

Glassfish ORB throws an exception saying that the timeouts need to "3 positive
decimal integers separated by :"

I traced this exception down to line #61 in TcpTimeoutsImpl
(http://mercurial2.foundry.sun.com/corba/corba-gfv2-master/file/0fc818f2f9e5/src/share/classes/com/sun/corba/se/impl/transport/TcpTimeoutsImpl.java)

The conditional states:
if ((data.length <= 3) || (data.length > 4))

I think the '(data.length <= 3)' should be '(data.length < 3)' to allow for 3 or
4 arguments.

Environment

Operating System: All
Platform: All

[glassfishrobot]

@glassfishrobot Commented
Reported by heartsjava

[glassfishrobot]

@glassfishrobot Commented
kcavanaugh said:
I have already fixed this in the GlassFish v3 CORBA version.
We'll be integrating that ORB into GFv3 probably in the next
month or two.

[glassfishrobot]

@glassfishrobot Commented
This issue was imported from java.net JIRA GLASSFISH_CORBA-2

[glassfishrobot]

• Issue Imported From: Inproper parsing of ORBTCPTimeouts javaee/glassfish-corba#2
• Original Issue Raised By:@glassfishrobot
• Original Issue Assigned To: @russgold

[pzygielo]

Seems to be fixed in 40b2f82