[#ICE-3488] Possible null value in group set of GroupAsyncRenderer

ICEfaces

Possible null value in group set of GroupAsyncRenderer

Created: 05/Sep/08 06:10 PM Updated: 27/Oct/08 03:46 PM

Component/s:

Framework

Affects Version/s:

1.7.1

Fix Version/s:

1.7.2

Environment:

ICEfaces 1.7.1

ICEsoft Forum Reference:

http://www.icefaces.org/JForum/posts/list/9619.page

Description

« Hide

When adding a session using the SessionRenderer.addCurrentSession(String) method, it's possible that a WeakReference containing null is added to the GroupAsyncRenderer's group set due to an HttpSession not being there yet. When requesting a render the null value is not an instance of Renderable nor HttpSession ending up at the final check to see if it's a PortletSession causing the ClassNotFoundException to be thrown in a non-portal environment as mentioned in the forum.

All

Comments

Change History

Subversion Commits

Sort Order:

[ Permlink | « Hide ]

Jack Van Ooststroom added a comment - 05/Sep/08 06:11 PM

Changed Fix Version(s) to 1.7.2

[ Permlink | « Hide ]

Jack Van Ooststroom added a comment - 05/Sep/08 06:24 PM

Now when invoking addCurrentSession(String) a new session is created if there is no current session yet. Marking this one as FIXED.

[ Permlink | « Hide ]

Jack Van Ooststroom added a comment - 08/Sep/08 03:43 PM

Changed the strategy a bit. A new session is not created anymore when a session does not exist during the addCurrentSession invocation. Instead an IllegalStateException is thrown as the addCurrentSession method should always be called from threads executing the JSF lifecycle. Additionally, warn level messages are logged when either addCurrentSession or removeCurrentSession is invoked and no session exists. Marking this one as FIXED again.

[ Permlink | « Hide ]

Jack Van Ooststroom added a comment - 09/Sep/08 12:28 PM

Reopening once again as the combination of CopyOnWriteArraySet and WeakReferences does not result in the WeakReferences being cleaned up automatically.

[ Permlink | « Hide ]

Jack Van Ooststroom added a comment - 09/Sep/08 01:13 PM

It was still possible getting null-values as after session clean-up the contained WeakReferences will point to null. They are not automatically removed from the CopyOnWriteArraySet. Therefore, on each render call whenever a WeakReference is encountered that points to null, it is removed from the Set. Later on we could consider a more pro-active solution to clean the Set of these kind of WeakReferences. Marking this one as FIXED once again.