prealloc

corazawaf · Oct 31, 2023 · f88163d · f88163d
1 parent e1a276c
commit f88163d
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/internal/corazawaf/transaction.go b/internal/corazawaf/transaction.go
@@ -563,8 +563,7 @@ func (tx *Transaction) GetField(rv ruleVariableParams) []types.MatchData {
 
 	// in the most common scenario filteredMatches length will be
 	// the same as matches length, so we avoid allocating per result
-	filteredMatches := make([]types.MatchData, len(matches))
-	filteredCount := 0
+	filteredMatches := make([]types.MatchData, 0, len(matches))
 
 	for _, c := range matches {
 		isException := false
@@ -576,11 +575,10 @@ func (tx *Transaction) GetField(rv ruleVariableParams) []types.MatchData {
 			}
 		}
 		if !isException {
-			filteredMatches[filteredCount] = c
-			filteredCount++
+			filteredMatches = append(filteredMatches, c)
 		}
 	}
-	matches = filteredMatches[:filteredCount]
+	matches = filteredMatches
 
 	if rv.Count {
 		count := len(matches)