Handled the annotation if it is null

subyssurendran666 · Oct 28, 2024 · b15d113 · b15d113
1 parent 4ab9cd9
commit b15d113
Show file tree

Hide file tree

Showing 2 changed files with 38 additions and 1 deletion.
diff --git a/...e.jdt.core.tests.compiler/src/org/eclipse/jdt/core/tests/dom/StandAloneASTParserTest.java b/...e.jdt.core.tests.compiler/src/org/eclipse/jdt/core/tests/dom/StandAloneASTParserTest.java
@@ -1929,5 +1929,39 @@ public void testBug568629() throws JavaModelException {
 			assertEquals("Syntax error, insert \"AssignmentOperator Expression\" to complete Expression",cu.getProblems()[6].getMessage());
 	}
 
+	public void testBugGithub2402() throws JavaModelException {
+        String contents = """
+        			package test;
+					public class TestClass {
+					    @com.Missing
+					    public TestClass() {
+					        new test.TestClass();
+					    }
+					}
+					class com {
+					}
+        		""";
+
+        ASTParser parser = ASTParser.newParser(AST_JLS_LATEST);
+        parser.setResolveBindings(true);
+        parser.setStatementsRecovery(true);
+        parser.setBindingsRecovery(true);
+        parser.setKind(ASTParser.K_COMPILATION_UNIT);
+        parser.setEnvironment(new String[0], new String[0], null, false);
+        parser.setSource(contents.toCharArray());
+        parser.setUnitName("test.TestClass");
+        ASTNode node = parser.createAST(null);
+        assertTrue("Should be a compilation unit", node instanceof CompilationUnit);
+        CompilationUnit cu = (CompilationUnit) node;
+        TypeDeclaration type = (TypeDeclaration) cu.types().get(0);
+        MethodDeclaration method = (MethodDeclaration) type.bodyDeclarations().get(0);
+        ExpressionStatement stmt = (ExpressionStatement) method.getBody().statements().get(0);
+        ClassInstanceCreation expr = (ClassInstanceCreation) stmt.getExpression();
+        IMethodBinding ctorBinding = expr.resolveConstructorBinding();
+        assertNotNull(ctorBinding);
+        IAnnotationBinding[] annotations = ctorBinding.getAnnotations();
+        assertNotNull(annotations);
+}
+
 
 }
diff --git a/org.eclipse.jdt.core/dom/org/eclipse/jdt/core/dom/MethodBinding.java b/org.eclipse.jdt.core/dom/org/eclipse/jdt/core/dom/MethodBinding.java
@@ -244,6 +244,9 @@ protected IAnnotationBinding[] filterTypeAnnotations(org.eclipse.jdt.internal.co
 			final boolean isConstructor = this.isConstructor();
 			for (int i = 0; i < length; i++) {
 				org.eclipse.jdt.internal.compiler.lookup.AnnotationBinding internalAnnotation = internalAnnotations[i];
+				if(internalAnnotation == null) {
+					continue;
+				}
 				final ReferenceBinding annotationType = internalAnnotation.getAnnotationType();
 				long metaTagBits = annotationType.getAnnotationTagBits();
 
@@ -612,7 +615,7 @@ protected JavaElement getUnresolvedJavaElement() {
 				return null;
 			}
 			if (!(this.resolver instanceof DefaultBindingResolver)) return null;
-	
+
 			DefaultBindingResolver defaultBindingResolver = (DefaultBindingResolver) this.resolver;
 			if (!defaultBindingResolver.fromJavaProject) return null;
 			ASTNode domNode = (ASTNode)defaultBindingResolver.bindingsToAstNodes.get(this);