Some checks produce results with file, but without sample. Make the c…

…heck more robust.
doyensec · Feb 8, 2021 · ba73cfd · ba73cfd
1 parent 9c705bd
commit ba73cfd
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/util/file.js b/src/util/file.js
@@ -113,9 +113,9 @@ export function writeIssues(root, isRelative, filename, result, isSarif){
               uri: issue.file !== "N/A" ? issue.file : "file:///"
             },
             region: {
-              startLine: issue.file !== "N/A" ? issue.location.line : 1,
-              startColumn: issue.file !== "N/A" ? issue.location.column + 1 : 1, // sarif columns start from 1
-              charLength: issue.file !== "N/A" ? issue.sample.length : 0
+              startLine: issue.location.line,
+              startColumn: issue.location.column + 1, // sarif columns start from 1
+              charLength: issue.sample ? issue.sample.length : 0
             }
           }
         }