On Tue, Oct 21, 2008 at 1:09 PM, Emmanuel Lecharny <[EMAIL PROTECTED]> wrote:
> I think there is something wrong in MINA code :
>
>   protected NioSession accept(IoProcessor<NioSession> processor,
>           ServerSocketChannel handle) throws Exception {
>
>       SelectionKey key = handle.keyFor(selector);
>       if (!key.isAcceptable()) {  <--- Here, if the key is null, you get a
> NPE...

While I agree with analysis that it will throw a NPE, the problem is
rather why we get a null key in Glassfish but not in any other
environment that I've tested in. Checking for null and returning null
might not solve the problem, only hide it.

> Could you check it with Glassfish when I will have committed the code ?

Sure, I'll check it tonight.

/niklas

Reply via email to