diff options
author | Andrei Selkin <andreyselkin@gmail.com> | 2015-11-05 23:21:56 +0300 |
---|---|---|
committer | Roman Ivanov <ivanov-jr@mail.ru> | 2015-11-05 20:59:34 -0800 |
commit | 8381754587bee0de49489e9bfb11e5912f664e87 (patch) | |
tree | ad3a1bf561ac8e5c25ebb79775e5f0af8e8f76c0 /src/main/java/com/puppycrawl/tools/checkstyle/checks/naming/ParameterNameCheck.java | |
parent | be8a60a4d95978e1b6cdb32fefca58f424e61178 (diff) | |
download | checkstyle-8381754587bee0de49489e9bfb11e5912f664e87.tar.gz |
Issue #2290: Fix NPE in isOverriddenMethod during validation of methods with implicit modifiers
Diffstat (limited to 'src/main/java/com/puppycrawl/tools/checkstyle/checks/naming/ParameterNameCheck.java')
-rw-r--r-- | src/main/java/com/puppycrawl/tools/checkstyle/checks/naming/ParameterNameCheck.java | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/src/main/java/com/puppycrawl/tools/checkstyle/checks/naming/ParameterNameCheck.java b/src/main/java/com/puppycrawl/tools/checkstyle/checks/naming/ParameterNameCheck.java index 9152f7063..3658dc4b4 100644 --- a/src/main/java/com/puppycrawl/tools/checkstyle/checks/naming/ParameterNameCheck.java +++ b/src/main/java/com/puppycrawl/tools/checkstyle/checks/naming/ParameterNameCheck.java @@ -118,11 +118,13 @@ public class ParameterNameCheck private static boolean isOverriddenMethod(DetailAST ast) { boolean overridden = false; final DetailAST parent = ast.getParent().getParent(); - final DetailAST annotation = parent.getFirstChild().getFirstChild(); - if (annotation.getType() == TokenTypes.ANNOTATION) { - final DetailAST overrideToken = annotation.findFirstToken(TokenTypes.IDENT); - if ("Override".equals(overrideToken.getText())) { - overridden = true; + if (parent.getFirstChild().getFirstChild() != null) { + final DetailAST annotation = parent.getFirstChild().getFirstChild(); + if (annotation.getType() == TokenTypes.ANNOTATION) { + final DetailAST overrideToken = annotation.findFirstToken(TokenTypes.IDENT); + if ("Override".equals(overrideToken.getText())) { + overridden = true; + } } } return overridden; |