Re: [BUG] net.jini.jeri.tcp.TcpServerEndpoint$LH$ServerConnectionImpl

Peter Jones <[email protected]>
Newsgroups gmane.comp.java.sun.jini
Message-ID <20070503152152.GA18635@east>
On Thu, May 03, 2007 at 08:34:04AM +0200, Mark Brouwer wrote:
> Peter Jones wrote:
>
>> For 6243057 (fixed in Porter) we added a workaround for 4720952 by
>> caching the result of socket.getInetAddress()-- see diffs appended
>> below-- but only for TcpServerEndpoint, and from your stack trace it
>> appears that you are now encountering the equivalent issue on the
>> client side.  I think that an equivalent workaround in TcpEndpoint
>> would be reasonable-- when the TcpEndpoint.ConnectionImpl constructor
>> is invoked the socket will not have been exposed to any code that
>> could close it-- although you would (just) need to cache the result of
>> socket.getRemoteSocketAddress() instead of socket.getInetAddress().
>
> Thanks Peter,
>
> I was also under the impression this was fixed but 6243057 was still
> listed as 'in progress' so that confused me and I didn't look into the
> code (maybe the issue should be closed). Will fix it at the client side
> and have a look at other usages of socket.getInetAddress() too.

Hmm, I don't know why 6243057 is listed as "in progress" on
bugs.sun.com-- its internal state is the appropriate "Fix Delivered",
which translates to "Closed" for other RFEs & bugs on bugs.sun.com.

-- Peter

--------------------------------------------------------------------------
Getting Started:     http://www.jini.org/wiki/Category:Getting_Started
Community Web Site:  http://jini.org
jini-users Archive:  http://archives.java.sun.com/archives/jini-users.html
Unsubscribing:       email "signoff JINI-USERS"  to [email protected]
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.