New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sonar validation failing with StackOverflowError #5175

Closed
romani opened this Issue Oct 7, 2017 · 1 comment

Comments

Projects
None yet
1 participant
@romani
Member

romani commented Oct 7, 2017

Build: https://travis-ci.org/checkstyle/checkstyle/jobs/284850599#L2087
but there a lot of other builds with same problem.
problem is not stable , as there builds wihtout problem - https://travis-ci.org/checkstyle/checkstyle/jobs/284859387#L2220

problematic file - InputDetailASTJustToMakeStackoverflowError.java

Sonar build item is disabled till problem is resolved - commit.
Bug to sonar need to be opened.
question: Why do we run sonar validation on our input files ? .... we should skip them.

log:

[INFO] 356/356 source files have been analyzed
[INFO] Java Main Files AST scan (done) | time=52134ms
[INFO] Java Test Files AST scan
[INFO] 1303 source files to be analyzed
[INFO] 293/1303 files analyzed, current file: 
/home/travis/build/checkstyle/checkstyle/src/test/java/com/puppycrawl/tools/checkstyle/checks/coding/ReturnCountCheckTest.java
[ERROR] A stack overflow error occured while analyzing file: 
/home/travis/build/checkstyle/checkstyle/src/test/resources/com/puppycrawl/tools/checkstyle/api/detailast/InputDetailASTJustToMakeStackoverflowError.java
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 06:00 min
[INFO] Finished at: 2017-10-07T23:36:31Z
[INFO] Final Memory: 145M/1336M
[INFO] ------------------------------------------------------------------------
---------------------------------------------------
constituent[0]: file:/usr/local/maven-3.5.0/conf/logging/
constituent[1]: file:/usr/local/maven-3.5.0/lib/aopalliance-1.0.jar
constituent[2]: file:/usr/local/maven-3.5.0/lib/cdi-api-1.0.jar
constituent[3]: file:/usr/local/maven-3.5.0/lib/commons-cli-1.4.jar
constituent[4]: file:/usr/local/maven-3.5.0/lib/commons-io-2.5.jar
constituent[5]: file:/usr/local/maven-3.5.0/lib/commons-lang3-3.5.jar
constituent[6]: file:/usr/local/maven-3.5.0/lib/guava-20.0.jar
constituent[7]: file:/usr/local/maven-3.5.0/lib/guice-4.0-no_aop.jar
constituent[8]: file:/usr/local/maven-3.5.0/lib/jansi-1.13.jar
constituent[9]: file:/usr/local/maven-3.5.0/lib/javax.inject-1.jar
constituent[10]: file:/usr/local/maven-3.5.0/lib/jcl-over-slf4j-1.7.22.jar
constituent[11]: file:/usr/local/maven-3.5.0/lib/jsr250-api-1.0.jar
constituent[12]: file:/usr/local/maven-3.5.0/lib/maven-artifact-3.5.0.jar
constituent[13]: file:/usr/local/maven-3.5.0/lib/maven-builder-support-3.5.0.jar
constituent[14]: file:/usr/local/maven-3.5.0/lib/maven-compat-3.5.0.jar
constituent[15]: file:/usr/local/maven-3.5.0/lib/maven-core-3.5.0.jar
constituent[16]: file:/usr/local/maven-3.5.0/lib/maven-embedder-3.5.0.jar
constituent[17]: file:/usr/local/maven-3.5.0/lib/maven-model-3.5.0.jar
constituent[18]: file:/usr/local/maven-3.5.0/lib/maven-model-builder-3.5.0.jar
constituent[19]: file:/usr/local/maven-3.5.0/lib/maven-plugin-api-3.5.0.jar
constituent[20]: file:/usr/local/maven-3.5.0/lib/maven-repository-metadata-3.5.0.jar
constituent[21]: file:/usr/local/maven-3.5.0/lib/maven-resolver-api-1.0.3.jar
constituent[22]: file:/usr/local/maven-3.5.0/lib/maven-resolver-connector-basic-1.0.3.jar
constituent[23]: file:/usr/local/maven-3.5.0/lib/maven-resolver-impl-1.0.3.jar
constituent[24]: file:/usr/local/maven-3.5.0/lib/maven-resolver-provider-3.5.0.jar
constituent[25]: file:/usr/local/maven-3.5.0/lib/maven-resolver-spi-1.0.3.jar
constituent[26]: file:/usr/local/maven-3.5.0/lib/maven-resolver-transport-wagon-1.0.3.jar
constituent[27]: file:/usr/local/maven-3.5.0/lib/maven-resolver-util-1.0.3.jar
constituent[28]: file:/usr/local/maven-3.5.0/lib/maven-settings-3.5.0.jar
constituent[29]: file:/usr/local/maven-3.5.0/lib/maven-settings-builder-3.5.0.jar
constituent[30]: file:/usr/local/maven-3.5.0/lib/maven-shared-utils-3.1.0.jar
constituent[31]: file:/usr/local/maven-3.5.0/lib/maven-slf4j-provider-3.5.0.jar
constituent[32]: file:/usr/local/maven-3.5.0/lib/org.eclipse.sisu.inject-0.3.3.jar
constituent[33]: file:/usr/local/maven-3.5.0/lib/org.eclipse.sisu.plexus-0.3.3.jar
constituent[34]: file:/usr/local/maven-3.5.0/lib/plexus-cipher-1.7.jar
constituent[35]: file:/usr/local/maven-3.5.0/lib/plexus-component-annotations-1.7.1.jar
constituent[36]: file:/usr/local/maven-3.5.0/lib/plexus-interpolation-1.24.jar
constituent[37]: file:/usr/local/maven-3.5.0/lib/plexus-sec-dispatcher-1.4.jar
constituent[38]: file:/usr/local/maven-3.5.0/lib/plexus-utils-3.0.24.jar
constituent[39]: file:/usr/local/maven-3.5.0/lib/slf4j-api-1.7.22.jar
constituent[40]: file:/usr/local/maven-3.5.0/lib/wagon-file-2.12.jar
constituent[41]: file:/usr/local/maven-3.5.0/lib/wagon-http-2.12-shaded.jar
constituent[42]: file:/usr/local/maven-3.5.0/lib/wagon-provider-api-2.12.jar
---------------------------------------------------
Exception in thread "main" java.lang.StackOverflowError
    at org.sonar.plugins.java.api.tree.BaseTreeVisitor.scan(BaseTreeVisitor.java:43)
    at org.sonar.plugins.java.api.tree.BaseTreeVisitor.visitBinaryExpression(BaseTreeVisitor.java:209)
    at org.sonar.java.resolve.TypeAndReferenceSolver.visitBinaryExpression(TypeAndReferenceSolver.java:675)
    at org.sonar.java.model.expression.BinaryExpressionTreeImpl.accept(BinaryExpressionTreeImpl.java:70)
    at org.sonar.plugins.java.api.tree.BaseTreeVisitor.scan(BaseTreeVisitor.java:43)

@romani romani added the approved label Oct 7, 2017

romani added a commit that referenced this issue Oct 7, 2017

romani added a commit that referenced this issue Oct 8, 2017

@romani romani closed this Oct 8, 2017

@romani romani added this to the 8.4 milestone Oct 8, 2017

@romani romani added the miscellaneous label Oct 8, 2017

timurt added a commit to timurt/checkstyle that referenced this issue Dec 19, 2017

timurt added a commit to timurt/checkstyle that referenced this issue Dec 19, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment