myfaces-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Christian Kaltepoth (JIRA)" <>
Subject [jira] Commented: (ORCHESTRA-41) NullPointerException in method findConversationContextId
Date Sat, 18 Jul 2009 14:22:14 GMT


Christian Kaltepoth commented on ORCHESTRA-41:

It seems like ORCHESTRA-34 wasn't fixed by adding a NPE check but by eliminating 
the cause of FrameworkAdapter.getCurrentInstance() returning null. 

See the SVN commit for ORCHESTRA-34:

Are you sure that this stack trace refers to Orchestra 1.3.1? 

An NPE seems possible with Orchestra 1.3 in line 140 of ConversationManager when
FrameworkAdapter.getCurrentInstance() returns null:

But in 1.3.1 line 140 only contains a source code comment:

> NullPointerException in method findConversationContextId
> --------------------------------------------------------
>                 Key: ORCHESTRA-41
>                 URL:
>             Project: MyFaces Orchestra
>          Issue Type: Bug
>          Components: Conversation, FrameworkAdapter
>    Affects Versions: 1.3.1
>         Environment: Windows XP SP2, Tomcat 6.0.20
>            Reporter: Bozhidar Bozhanov
>   Original Estimate: 0.08h
>  Remaining Estimate: 0.08h
> After some time, (probably when a thread times-out) the following appears:
> Exception in thread "org.apache.myfaces.orchestra.conversation.ConversationWiperThread"
> 	at org.apache.myfaces.orchestra.conversation.ConversationManager.findConversationContextId(
> 	at org.apache.myfaces.orchestra.conversation.ConversationManager.removeAndInvalidateConversationContext(
> 	at org.apache.myfaces.orchestra.conversation.ConversationManager.checkTimeouts(
> 	at org.apache.myfaces.orchestra.conversation.ConversationWiperThread._run(
> 	at
> It doesn't bring any trouble to the front-end, but still, it is an exception :)
> The problem, I think is that the ThreadLocal variable (after the thread has timed-out)
return null, so no conversationContext anymore. A little anti-NPE check in the findConversaionContextId
would get rid of the exception.
> I'm not sure whether this happens in 1.3.1, I checked the source code in the repository
and there was no NPE check, so I presume the issue is still there.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message