fix NullAway warnings

uber · Aug 4, 2023 · 6cc3e83 · 6cc3e83
1 parent 95df551
commit 6cc3e83
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/nullaway/src/main/java/com/uber/nullaway/ErrorBuilder.java b/nullaway/src/main/java/com/uber/nullaway/ErrorBuilder.java
@@ -203,8 +203,10 @@ private Description.Builder addSuggestedSuppression(
         if (config.getCastToNonNullMethod() != null && canBeCastToNonNull(suggestTree)) {
           builder = addCastToNonNullFix(suggestTree, builder);
         } else {
-          builder =
-              addSuppressWarningsFix(suppressibleNode(state.getPath()), builder, suppressionName);
+          Tree suppressibleNode = suppressibleNode(state.getPath());
+          if (suppressibleNode != null) {
+            builder = addSuppressWarningsFix(suppressibleNode, builder, suppressionName);
+          }
         }
         break;
       case CAST_TO_NONNULL_ARG_NONNULL:
@@ -343,7 +345,8 @@ private boolean canBeCastToNonNull(Tree tree) {
         // make the parameter @NonNull and add casts at call sites, but that is beyond the scope of
         // our suggested fixes
         Symbol symbol = ASTHelpers.getSymbol(tree);
-        return !(symbol.getKind().equals(ElementKind.PARAMETER)
+        return !(symbol != null
+            && symbol.getKind().equals(ElementKind.PARAMETER)
             && hasNullableAnnotation(symbol, config));
       default:
         return true;