Populate the fixes only when autofix is not empty (#1226)

Change-Id: If4de66d1ea0fd5a179808d023fdac677437c6d5a Signed-off-by: Cosmin Cojocar <[email protected]>
securego · Sep 18, 2024 · be8bd6e · be8bd6e
1 parent 3004932
commit be8bd6e
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/report/sarif/builder.go b/report/sarif/builder.go
@@ -80,20 +80,23 @@ func NewTool(driver *ToolComponent) *Tool {
 
 // NewResult instantiate a Result
 func NewResult(ruleID string, ruleIndex int, level Level, message string, suppressions []*Suppression, autofix string) *Result {
-	return &Result{
+	result := &Result{
 		RuleID:       ruleID,
 		RuleIndex:    ruleIndex,
 		Level:        level,
 		Message:      NewMessage(message),
 		Suppressions: suppressions,
-		Fixes: []*Fix{
+	}
+	if len(autofix) > 0 {
+		result.Fixes = []*Fix{
 			{
 				Description: &Message{
 					Markdown: autofix,
 				},
 			},
-		},
+		}
 	}
+	return result
 }
 
 // NewMessage instantiate a Message