63969 – Stackoverflow in JSF

Bug 63969 - Stackoverflow in JSF

Summary: Stackoverflow in JSF

Status:	RESOLVED FIXED

Alias:	None

Product:	Tomcat 9
Classification:	Unclassified
Component:	Catalina (show other bugs)
Version:	9.0.29
Hardware:	PC All

Importance:	P2 regression (vote)
Target Milestone:	-----
Assignee:	Tomcat Developers Mailing List

URL:
Keywords:

Duplicates (1):	63976 (view as bug list)
Depends on:
Blocks:

Reported:	2019-11-27 15:09 UTC by ekkelenkamp
Modified:	2019-12-15 22:03 UTC (History)
CC List:	2 users (show)

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description ekkelenkamp 2019-11-27 15:09:36 UTC

When upgrading from Tomcat 9.0.27 to 9.0.29 the following stack overflow occurs:

java.lang.StackOverflowError
		at java.logging/java.util.logging.LogRecord$CallerFinder.get(LogRecord.java:709)
		at java.logging/java.util.logging.LogRecord.inferCaller(LogRecord.java:683)
		at java.logging/java.util.logging.LogRecord.getSourceMethodName(LogRecord.java:383)
		at org.apache.juli.AsyncFileHandler.publish(AsyncFileHandler.java:112)
		at java.logging/java.util.logging.Logger.log(Logger.java:979)
		at java.logging/java.util.logging.Logger.doLog(Logger.java:1006)
		at java.logging/java.util.logging.Logger.log(Logger.java:1117)
		at com.sun.faces.application.ApplicationImpl.newThing(ApplicationImpl.java:1735)
		at com.sun.faces.application.ApplicationImpl.createComponentApplyAnnotations(ApplicationImpl.java:1896)
		at com.sun.faces.application.ApplicationImpl.createComponent(ApplicationImpl.java:1166)
		at com.sun.faces.facelets.tag.jsf.ComponentTagHandlerDelegateImpl.createComponent(ComponentTagHandlerDelegateImpl.java:602)
		at com.sun.faces.facelets.tag.jsf.ComponentTagHandlerDelegateImpl.apply(ComponentTagHandlerDelegateImpl.java:175)
		at javax.faces.view.facelets.DelegatingMetaTagHandler.apply(DelegatingMetaTagHandler.java:120)
		at javax.faces.view.facelets.CompositeFaceletHandler.apply(CompositeFaceletHandler.java:95)
		at com.sun.faces.facelets.compiler.NamespaceHandler.apply(NamespaceHandler.java:93)
		at com.sun.faces.facelets.compiler.EncodingHandler.apply(EncodingHandler.java:87)
		at com.sun.faces.facelets.impl.DefaultFacelet.include(DefaultFacelet.java:312)
		at com.sun.faces.facelets.impl.DefaultFacelet.include(DefaultFacelet.java:371)
		at com.sun.faces.facelets.impl.DefaultFacelet.include(DefaultFacelet.java:350)
		at com.sun.faces.facelets.impl.DefaultFaceletContext.includeFacelet(DefaultFaceletContext.java:199)
		at com.sun.faces.facelets.tag.ui.IncludeHandler.apply(IncludeHandler.java:124)
		at com.sun.faces.facelets.tag.ui.InsertHandler.apply(InsertHandler.java:116)
		at javax.faces.view.facelets.DelegatingMetaTagHandler.applyNextHandler(DelegatingMetaTagHandler.java:137)
		at com.sun.faces.facelets.tag.jsf.ComponentTagHandlerDelegateImpl.apply(ComponentTagHandlerDelegateImpl.java:202)
		at javax.faces.view.facelets.DelegatingMetaTagHandler.apply(DelegatingMetaTagHandler.java:120)
		at javax.faces.view.facelets.CompositeFaceletHandler.apply(CompositeFaceletHandler.java:95)
		at com.sun.faces.facelets.compiler.NamespaceHandler.apply(NamespaceHandler.java:93)
		at com.sun.faces.facelets.compiler.EncodingHandler.apply(EncodingHandler.java:87)
		at com.sun.faces.facelets.impl.DefaultFacelet.include(DefaultFacelet.java:312)
		at com.sun.faces.facelets.impl.DefaultFacelet.include(DefaultFacelet.java:371)
		at com.sun.faces.facelets.impl.DefaultFacelet.include(DefaultFacelet.java:350)
		at com.sun.faces.facelets.impl.DefaultFaceletContext.includeFacelet(DefaultFaceletContext.java:199)
		at com.sun.faces.facelets.tag.ui.IncludeHandler.apply(IncludeHandler.java:124)
		at com.sun.faces.facelets.tag.ui.InsertHandler.apply(InsertHandler.java:116)
		at javax.faces.view.facelets.DelegatingMetaTagHandler.applyNextHandler(DelegatingMetaTagHandler.java:137)
		at com.sun.faces.facelets.tag.jsf.ComponentTagHandlerDelegateImpl.apply(ComponentTagHandlerDelegateImpl.java:202)

Comment 1 Mark Thomas 2019-11-27 17:08:17 UTC

I think this is fixed already. Can you try a 9.0.30 dev build from here and report back?
http://people.apache.org/~markt/dev/v9.0.30-dev/

Note that is a dev build for testing only. It isn't an official release. Please don't use it in production and please don't blame me if your server catches fire when you run it ;)

Comment 2 Mark Thomas 2019-11-28 16:18:50 UTC

*** Bug 63976 has been marked as a duplicate of this bug. ***

Comment 3 Patrick WENDJI 2019-11-29 12:30:36 UTC

Hello,

Has the fix been ported to 8.5.x branch?

Comment 4 Mark Thomas 2019-11-29 12:58:58 UTC

8.5.50-dev build - same caveats apply as for the 9.0.x dev build

Comment 5 Conrad Kostecki 2019-12-02 10:56:16 UTC

(In reply to Mark Thomas from comment #4)
> 8.5.50-dev build - same caveats apply as for the 9.0.x dev build

Yup, I can confirm, it seems to be fixed in 8.5.50-dev.

Comment 6 Mark Thomas 2019-12-02 13:48:44 UTC

Thanks for the confirmation.

Fixed in:
- master for 9.0.30 onwards
- 8.5.x for 8.5.50 onwards

Comment 7 ekkelenkamp 2019-12-15 22:03:20 UTC

I tested with 9.0.30 and can confirm the bug is indeed fixed. Thanks for that!