🐛 fix case where when SARIF is parsed for the first time, it fails if…

… the results list for a run is null rather than empty
pixee · Nov 30, 2023 · 62900b4 · 62900b4
1 parent 9e94732
commit 62900b4
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/framework/codemodder-base/src/main/java/io/codemodder/DefaultSarifParser.java b/framework/codemodder-base/src/main/java/io/codemodder/DefaultSarifParser.java
@@ -71,11 +71,14 @@ private Stream<Map.Entry<String, RuleSarif>> fromSarif(
         ServiceLoader.load(RuleSarifFactory.class).stream()
             .map(ServiceLoader.Provider::get)
             .collect(Collectors.toUnmodifiableList());
+    final var runResults = run.getResults();
     final var allResults =
-        run.getResults().stream()
-            .map(result -> extractRuleId(result, run))
-            .filter(Objects::nonNull)
-            .distinct();
+        runResults != null
+            ? runResults.stream()
+                .map(result -> extractRuleId(result, run))
+                .filter(Objects::nonNull)
+                .distinct()
+            : Stream.<String>empty();
 
     return allResults.flatMap(
         rule -> tryToBuild(toolName, rule, sarif, repositoryRoot, factories).stream());