Skip to content

Commit 03d1aa6

Browse files
committed
GROOVY-11522: fix for NPE
1 parent 3306448 commit 03d1aa6

1 file changed

Lines changed: 3 additions & 4 deletions

File tree

src/main/java/org/codehaus/groovy/classgen/VariableScopeVisitor.java

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -328,8 +328,7 @@ private void checkPropertyOnExplicitThis(final PropertyExpression expression) {
328328
String name = expression.getPropertyAsString();
329329
if (name == null || name.equals("class")) return;
330330
Variable member = findClassMember(currentClass, name);
331-
if (member == null) return;
332-
checkVariableContextAccess(member, expression);
331+
if (member != null) checkVariableContextAccess(member, expression);
333332
}
334333

335334
private void checkVariableContextAccess(final Variable variable, final Expression expression) {
@@ -555,9 +554,9 @@ public void visitDeclarationExpression(final DeclarationExpression expression) {
555554
@Override
556555
public void visitFieldExpression(final FieldExpression expression) {
557556
String name = expression.getFieldName();
558-
//TODO: change that to get the correct scope
557+
// TODO: change that to get the correct scope
559558
Variable variable = findVariableDeclaration(name);
560-
checkVariableContextAccess(variable, expression);
559+
if (variable != null) checkVariableContextAccess(variable, expression);
561560
}
562561

563562
@Override

0 commit comments

Comments
 (0)