[MYFACES-3585] FacesContextFactoryImpl is not released in FacesContextImpl.release()

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.1.8
• Fix Version/s: 2.0.15, 2.1.9
• Assignee: Leonardo Uribe
• Reporter: Pavel Arnošt
• Created: 18/Jul/12 07:23
• Updated: 24/Sep/12 16:29
• Resolved: 23/Jul/12 12:11

Description

There is a bug in release method of FacesContextImpl class:

@Override
public final void release()
{
assertNotReleased();

_messages = null;
_orderedMessages = null;
_currentPhaseId = null;
_responseStream = null;
_responseWriter = null;
_maximumSeverity = null;
_partialViewContext = null;
_facesContextFactory = null;
_renderKitFactory = null;
_partialViewContextFactory = null;

if (_facesContextFactory != null)

{ _facesContextFactory.release(); _facesContextFactory = null; }

_facesContextFactory is set to null and then tested for null. _facesContextFactory is neved released then.

Issue Links

• is duplicated by: MYFACES-3615 Memory Leak related to ThreadLocal with org.apache.myfaces.context.servlet.ServletExternalContextImpl