Don't scan into nested enums in ClassInitializationDeadlock

google#4378 PiperOrigin-RevId: 629452163 (cherry picked from commit 6a8f493)
PicnicSupermarket · Apr 30, 2024 · 5a3c54f · 5a3c54f
1 parent 54148b6
commit 5a3c54f
Show file tree

Hide file tree

Showing 2 changed files with 31 additions and 1 deletion.
diff --git a/core/src/main/java/com/google/errorprone/bugpatterns/ClassInitializationDeadlock.java b/core/src/main/java/com/google/errorprone/bugpatterns/ClassInitializationDeadlock.java
@@ -71,7 +71,7 @@ public Description matchClass(ClassTree tree, VisitorState state) {
       @Override
       public Void visitClass(ClassTree node, Void unused) {
         for (Tree member : node.getMembers()) {
-          if (member.getKind().equals(Tree.Kind.CLASS)) {
+          if (member instanceof ClassTree) {
             continue;
           }
           scan(member, null);

diff --git a/core/src/test/java/com/google/errorprone/bugpatterns/ClassInitializationDeadlockTest.java b/core/src/test/java/com/google/errorprone/bugpatterns/ClassInitializationDeadlockTest.java
@@ -266,4 +266,34 @@ public void negativeNonPrivateUnrelatedSuper() {
             "}")
         .doTest();
   }
+
+  @Test
+  public void nestedEnum() {
+    testHelper
+        .addSourceLines(
+            "TestInterface.java", //
+            "public interface TestInterface {",
+            "  default Object foo() {",
+            "    return null;",
+            "  }",
+            "  enum TestEnum implements TestInterface {",
+            "    INSTANCE;",
+            "  }",
+            "}")
+        .doTest();
+  }
+
+  @Test
+  public void nestedInterface() {
+    testHelper
+        .addSourceLines(
+            "Foo.java", //
+            "interface Foo {",
+            "  default void foo() {}",
+            "  interface Sub extends Foo {",
+            "    final Sub INSTANCE = new Sub() {};",
+            "  }",
+            "}")
+        .doTest();
+  }
 }