SONARJAVA-1588 Log file causing SOError

Author: benzonico

java-frontend/src/main/java/org/sonar/java/ast/JavaAstScanner.java

@@ -91,6 +91,9 @@ private void simpleScan(File file) {
     } catch (Exception e) {
       checkInterrrupted(e);
       throw new AnalysisException(getAnalyisExceptionMessage(file), e);
+    } catch (StackOverflowError error) {
+      LOG.error("A stack overflow error occured while analyzing file: " + file.getAbsolutePath());
+      throw error;
     }
   }
 

java-frontend/src/test/java/org/sonar/java/ast/JavaAstScannerTest.java

@@ -161,10 +161,25 @@ public void should_propagate_visitor_exception_when_no_parse_error() {
     scanner.scan(ImmutableList.of(new File("src/test/resources/AstScannerNoParseError.txt")));
   }
 
+  @Test
+  public void should_propagate_SOError() {
+    thrown.expect(StackOverflowError.class);
+    JavaAstScanner scanner = defaultJavaAstScanner();
+    scanner.setVisitorBridge(new VisitorsBridge(new CheckThrowingSOError()));
+    scanner.scan(ImmutableList.of(new File("src/test/resources/AstScannerNoParseError.txt")));
+  }
+
   private static JavaAstScanner defaultJavaAstScanner() {
     return new JavaAstScanner(new ActionParser<Tree>(Charsets.UTF_8, FakeLexer.builder(), FakeGrammar.class, new FakeTreeFactory(), new JavaNodeBuilder(), FakeLexer.ROOT));
   }
 
+  private static class CheckThrowingSOError implements JavaFileScanner {
+
+    @Override
+    public void scanFile(JavaFileScannerContext context) {
+      throw new StackOverflowError();
+    }
+  }
   private static class CheckThrowingException implements JavaFileScanner {
 
     private final RuntimeException exception;