fix: Add copyString function to symbolizer to avoid retaining memory (#…

…13146)
grafana · Jun 6, 2024 · 86b119a · 86b119a
1 parent 1b80458
commit 86b119a
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/pkg/chunkenc/symbols.go b/pkg/chunkenc/symbols.go
@@ -6,6 +6,7 @@ import (
 	"fmt"
 	"hash"
 	"io"
+	"strings"
 	"sync"
 
 	"github.com/pkg/errors"
@@ -78,6 +79,7 @@ func (s *symbolizer) add(lbl string) uint32 {
 
 	idx, ok = s.symbolsMap[lbl]
 	if !ok {
+		lbl = strings.Clone(lbl)
 		idx = uint32(len(s.labels))
 		s.symbolsMap[lbl] = idx
 		s.labels = append(s.labels, lbl)