[Quickfixj-users] Question about : Race Condition/NPE on Session.disconnect()

[gu h. <hai...@ho...>]

Hi, all

Recently I get the same error as bug QFJ-211, and I agree with Steve Bate 
and Toli Kuznets that the reason is race condition. But I don't agree with 
the solution. Cause "Modified Session.disconnect to eliminate the race 
condition." only solve the NPE problem. If in class AbstractIoHandler 
sessionClosed()'s quickFixSession.disconnect() is executed before in the 
Session class nextLogout()'s disconnect(), then the disconnect() will be 
executed twice. And the application.onLogout() will be executed twice.(I 
have seen it happens) That's a bug and should be avoid. I am thinking if we 
just delete the following code in AbstractIoHandler sessionClosed(), I 
think the problem will be solved.
             if (quickFixSession.hasResponder()) {
                quickFixSession.disconnect();
            }

how do you think that? Am I understood?

regards 
mike

_________________________________________________________________
与联机的朋友进行交流，请使用 MSN Messenger:  http://messenger.msn.com/cn